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

Add Required Tests to Branch Protection Rules #10

Open
ryanmrichard opened this issue Jul 20, 2022 · 0 comments
Open

Add Required Tests to Branch Protection Rules #10

ryanmrichard opened this issue Jul 20, 2022 · 0 comments

Comments

@ryanmrichard
Copy link
Member

In theory every repo in the NWChemEx-Project organization has branch protection rules setup for the master branch (requires PRs, PRs need reviews, etc.). At one point we also specified via the GitHub API which CI workflows were required to pass before the PR is merged (for the record, all of the CI workflows need to pass before a PR should be merged). By specifying the required workflows as part of the branch protection rules, GitHub prevents us from accidentally merging a PR with failing workflows. Right now all of the workflows run, but if you're not careful, you can actually merge the PR with failing workflows.

In setting the required workflows, GitHub uses the names of the workflows. The problem previously was that somehow the names changed and so GitHub was waiting forever for non-existent workflows to run. To avoid the PRs being in limbo we dropped the required workflows. This issue is for resetting the branch protection rules so that they enforce the workflows passing using their current names (and hopefully figuring out a way to future proof the names so we don't have to go through this again).

@twindus twindus transferred this issue from NWChemEx-Project/DeveloperTools Aug 1, 2022
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

1 participant