-
Notifications
You must be signed in to change notification settings - Fork 163
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
Move back to latest
for goreleaser
#668
Comments
Are we sure we want to move back to latest instead of upgrading the version periodically? Manually upgrading would prevent the CI unexpectedly breaking on main like it did twice already. |
Could we automate the bumping? For other stuff like Github Actions we use dependabot/renovate bot, but here I don't think there is an equivalent tool that is ready for this. Maybe we can ask upstream on the goreleaser repositories for advice? |
it would be possible with renovate i believe. maybe with some additional config to cover all occurrences, but it should be possible |
@mowies Do you have any links/examples explaining how this would be done with renovate? |
Something like this should work with renovate. It's a custom regex manager that matches the goreleaser action's input for the version. It could probably be done a little nicer and more resilient but it should work I think.
|
@mx-psi if you could take care of setting up renovate in this repo (or tell me what to do), then i'd be happy to contribute the needed config changes :) |
I believe you will have to file an issue similar to this one: open-telemetry/community/issues/1792 but for this repository. I am happy to voice my support in such an issue. |
Cool! I filed the issue: open-telemetry/community#2403 |
i will work on this when the renovate PR is merged, pls assign to me :) |
Done! |
PR is open for this: #728 |
seems like the regex I added still has some issues. I'll do a follow up PR |
Was this fixed by #733? |
unfortunately not, but i pivoted to test everything on my own fork with renovate activated there, so the next one will stick 😅 |
Ok I think I fixed it now :) |
Because of goreleaser/goreleaser/issues/5138, CI is currently broken when using goreleaser v2.3.0+.
#667 fixes this temporarily for the next release
The text was updated successfully, but these errors were encountered: