search

ACCESS-Cloud-Based-InSAR / dem-stitcher

Download and merge DEM tiles
Apache License 2.0
39 stars 15 forks source link

Update branch protection to use Rulesets #92

Closed jhkennedy closed 1 week ago

jhkennedy commented 1 week ago

With branch protection rules, when there are required checks for merging into dev: image

The release workflow will not be able to fast-forward main->dev to bring the release tag into the main branch, and a PR like #91 will be opened instead.

Notably, eventhough we allow an automation user to bypass a pull request image

this has no effect on required status checks: https://github.com/orgs/community/discussions/43460)

Rulesets allows more fine-grained control and would allow teams to bypass required status checks: https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-rulesets/about-rulesets#about-rulesets

jhkennedy commented 1 week ago

For now, I've disabled the required checks on dev so that the automation will work in the future, but I disallowed everyone, except the automation user, from pushing to dev.