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

Protect the main branch #128

Open
bebatut opened this issue Jun 25, 2024 · 7 comments
Open

Protect the main branch #128

bebatut opened this issue Jun 25, 2024 · 7 comments

Comments

@bebatut
Copy link
Member

bebatut commented Jun 25, 2024

If we do that, the Action can not update tool and tutorial lists

Possible solution

  • Protect main and have PR with changes instead of pushing directly

    But someone needs to merge the PR every week

  • Allow only the CI to push directly

    Not sure how to do that

  • Store the results in a different repo (as the research-software-ecosystem does)

@nsoranzo
Copy link
Member

* Store the results in a different repo (as the research-software-ecosystem does)

You can also store the results in a different branch of this repo (I do it in a non-Galaxy-related repo).

@bebatut
Copy link
Member Author

bebatut commented Jun 25, 2024

I like @nsoranzo's idea!

@paulzierep
Copy link
Collaborator

I agree that would work, but is it not a somewhat strange way of using git ? :)

@paulzierep
Copy link
Collaborator

I did re-protect the main branch again, btw., I only unprotected to show in the demo how the CI is working.

@nsoranzo
Copy link
Member

@paulzierep
Copy link
Collaborator

@nsoranzo it seems that storing the results in a different branch does bring some new issues: https://github.com/galaxyproject/galaxy_codex/actions/runs/10338185162/job/28615936668
Maybe you have an idea how to handle this ?

@paulzierep
Copy link
Collaborator

I think that should do the trick, #163

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

No branches or pull requests

3 participants