-
Notifications
You must be signed in to change notification settings - Fork 1
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
Repo Changes Breaking Runiac Build #5
Comments
@tiny-dancer, I have not updated anything recently, but I can take a look. |
It looks like @PseudoCoding added branch protection on this repo on Jan 24. What are you doing with regards to release automation where you're either not going through PRs or aren't having them approved? |
The update to the homebrew file in this repo is entirely performed via goreleaser, a release automation library for golang. |
https://goreleaser.com/customization/homebrew/ Ideally we wouldn’t add a human layer into an automated action. |
Is it being run under an admin user? If that's the case, we can just uncheck the box to include admins in branch protection. |
Good question, I think it was a token you provided? If you could provide a new token that matches admin can update accordingly |
That's an admin token. Just unchecked |
@amyschoen FYI the build still failed |
@amyschoen - I'm trying to get runiac development going again since MGs departure. I've got our release pipeline working now and I'm trying to troubleshoot the homebrew file automation that @tiny-dancer mentioned above. Can you verify if the token is still valid? I'm not seeing any obvious errors in the goreleaser-cli job indicating that this is failing. Unfortunately, its been a while since that last successful release ( |
I'm guessing you're referring to this workflow @mkrouse: https://github.com/Optum/runiac/blob/main/.github/workflows/release.yml I see references to The second one doesn't exist. The first and third ones are local and might have stopped working as expected when folks left. I suggest using the secrets set up at the org level - |
@amyschoen I had this working on a previous release for runiac, the last release failed again, looks like branch permissions were added back, I did enable it via the PR route but looks to be a lot more steps which goes against the automation. |
If there was a change recently to require pull requests and approval for this repo it has broken runiac release automation:
https://github.com/Optum/runiac/runs/5345432758?check_suite_focus=true
The text was updated successfully, but these errors were encountered: