-
Notifications
You must be signed in to change notification settings - Fork 10
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
Design and implement a better release workflow #15
Comments
Just to make sure I understand the goal here:
That all makes sense to me, but I just want to kinda confirm that's the intended flow here. The naive idea I'd have for this would be to have branch protection on, but have an exception for the ngrok-ci bot, allowing it to push during the release flow. I guess the reason we might not want to do that is because we want to use github actions? Is that the thing that makes that an undesirable solution? |
Yeah, you pretty much got the process down. We use the Maven release plugin which coordinates and executes these steps. I'd say we're loosely following semver in general. We'll likely have guidelines more refined when
Yeah, I think you're tracking correctly. We have branch protection on, and we attempted to allow for an exception for CI to bypass said restrictions. However, as I cited above, there's not a great built-in bypass for GitHub Actions specifically. The current flow involves manually disabling branch protection to allow for GitHub Actions runners to successfully make a release commits + tag. |
We are currently unable to bypass our
main
branch protection, which prevents our Release CI from pushing version number updates to our POMs. This requires some undesirable workarounds in order to perform a release via CI.This seems to be a highly requested and discussed feature that has been lacking support from GitHub Actions for a while now.
Goals:
main
as a protected branchThe text was updated successfully, but these errors were encountered: