-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
ci: bring bazel ci up to parity with checks performed in old ci #74557
Conversation
Up until this point, checking the release justification and whether all generated files were up to date was done only in GitHub CI rather than Bazel CI. We just copy those same checks into Bazel CI as well. Closes cockroachdb#68385. Release note: None
766eb63
to
5ffe06c
Compare
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.
Could you direct me to what's being changed in the build logs? ;) [I'm not familiar enough with the bazel and old ci checks to know where to look / what I should be looking for?] Thanks!
A read of the code itself makes senses to me!
Reviewed 7 of 7 files at r1, all commit messages.
Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @rail)
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.
approving, since code itself makes sense to me
Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @rail)
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.
LGTM,
Sorry -- in the build configuration link I sent in that comment, everything is new. It's a fully new build configuration. |
bors r=rail,celiala |
Build succeeded: |
Up until this point, checking the release justification and whether all
generated files were up to date was done only in GitHub CI rather than
Bazel CI. We just copy those same checks into Bazel CI as well.
Closes #68385.
Release note: None