Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always mirror full repository #1195

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

Eskaan
Copy link
Contributor

@Eskaan Eskaan commented Dec 4, 2023

Description

I noticed that the mirror workflow can't run on pull requests due to unavailability of secrets, so I changed it to run whenever we push a branch in the repo and always clone the entire repository using --mirror (/refs/*, includes all pull requests, branches and tags).

Type of change

  • Development change (no change visible to user)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
    • The pull requests are now mirrored under /refs/pulls/.../head instead of /refs/heads/pull/...
  • Documentation only update (no change to the factual codebase)
  • This change requires a documentation update

Checklist:

  • untested, but is expected to work (tested the commands in my terminal)

@Eskaan
Copy link
Contributor Author

Eskaan commented Dec 5, 2023

@VuiMuich I would suggest pushing this to a branch again to ensure it's working.

@VuiMuich
Copy link
Member

VuiMuich commented Dec 5, 2023

Probably won't have time before Thursday.

@Eskaan
Copy link
Contributor Author

Eskaan commented Feb 5, 2024

@VuiMuich reminder this exists. 🙂

@mautamu
Copy link
Member

mautamu commented Jun 1, 2024

@Eskaan For testing purposes, we can just draw this commit into a temporary testing branch and then create a PR against that branch to confirm it's working properly, correct? Any other tests required?

@Eskaan
Copy link
Contributor Author

Eskaan commented Jun 1, 2024

Yes, that is what I've been waiting for. If you put this in it's own testing branch, it will get access to this repositories secrets and can do a live-test without being on the main branch.

@mautamu
Copy link
Member

mautamu commented Jun 1, 2024

@Eskaan
Copy link
Contributor Author

Eskaan commented Jun 2, 2024

About #1265 and #1266: The repository will only be updated once we add a commit onto one of the repositories branches. But, it will always mirror the full repository over to codeberg.
That is because a ci run inside a pull request doesn't get access to the repositories secrets.

@Eskaan
Copy link
Contributor Author

Eskaan commented Jun 2, 2024

Honestly, I am unsure why Codeberg marked those (and only those) as internal references, as there were never any pull requests made on that repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants