-
Notifications
You must be signed in to change notification settings - Fork 360
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
[email protected] #2413
[email protected] #2413
Conversation
Hello @zhangskz, @mkruskal-google, @googleberg, modules you maintain (protobuf) have been updated in this PR. Please review the changes. |
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.
Hello @bazelbuild/bcr-maintainers, all modules in this PR have been approved by their maintainers. Please take a final look to merge this PR.
CI is failing |
Require module maintainers' approval for newly pushed changes.
e3db9de
to
8f0df53
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.
Hello @bazelbuild/bcr-maintainers, all modules in this PR have been approved by their maintainers. Please take a final look to merge this PR.
@zhangskz CI is still failing on Bazel 6 |
Thanks, will likely remove the rust presubmit entirely for now as the failing target is not actually intended for release / widespread use yet (and we are thus missing the equivalent test coverage in our repo). Will update this PR shortly once the equivalent change is made to protobuf repo. |
Rust protobuf is in development and not intended for use / released yet. We should add this back at some point when rust is ready for release, after adding equivalent GHA presubmit tests.
8f0df53
to
686f5ea
Compare
Require module maintainers' approval for newly pushed changes.
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.
Hello @bazelbuild/bcr-maintainers, all modules in this PR have been approved by their maintainers. Please take a final look to merge this PR.
Release: https://github.com/protocolbuffers/protobuf/releases/tag/v28.0-rc1
Automated by Publish to BCR