-
Notifications
You must be signed in to change notification settings - Fork 14
Update repo-conf #295
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
Update repo-conf #295
Conversation
Marenz
commented
Apr 2, 2025
- Update repo-config to 0.12.3
- Update to repo-config 0.13.1
Signed-off-by: Mathias L. Baumann <[email protected]>
|
WIP, ci is failing with stuff such as |
|
Actually, CI is failing differenttly XD but my bug above is also a problem |
|
Correction, apparently the quoted error is just locally, where as the CI is a protolint not found issue |
|
I suspect someone needs to update the reuqired ci check settings? I don't have access, @llucax maybe? |
Signed-off-by: Mathias L. Baumann <[email protected]>
|
Will do after the review |
|
Oh, damn, you forgot to bump the dependencies to v0.13.1 in |
|
Me? You mean the migration script! 😆 |
Yeah, that could be an addition to the migration script, I also found myself saying "why am I doing this manually" :D At first I thought about the migration script to be something to run on top of the dependabot PR, but since the dependabot PR had 15 other updates in it (which should be solved now), I started to do a separate PR for this upgrade. Hopefully for the next upgrade we can just push the migration script results to the dependabot PR and that it. In a not so distant future we can have a github action running the migration script for us and showing any manual steps in the action run summary, so can review it and only have to push the manual fixes, and then we get the github action using a LLM to actually do the manual steps for us 🤖 |