-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Prevent metadata changes in a stable branch #48994
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shipit
Hi @sivel, thank you for submitting this pull-request! |
thanks @sivel this will save us from ourselves |
@sivel, I assume we have override privileges to still merge if we have had a business agreement on this? |
@thaumos yes. It's just a CI failure, which doesn't prevent us from merging. We just have to accept that it failed, and if it failed only for that reason, we could merge it. CI should only fail in the PR state, and not after merge. |
Also, fwiw, this only fails on PRs with a stable base branch. This does not apply to devel. |
(cherry picked from commit 7287d39) Co-authored-by: Matt Martz <matt@sivel.net>
…). (cherry picked from commit 7287d39) Co-authored-by: Matt Martz <matt@sivel.net>
SUMMARY
Prevent metadata changes in a stable branch
ISSUE TYPE
COMPONENT NAME
test/sanity/validate-modules/main.py
ADDITIONAL INFORMATION