Skip to content
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

Merged

Conversation

publish-to-bcr-bot[bot]
Copy link
Contributor

@bazel-io
Copy link
Member

Hello @zhangskz, @mkruskal-google, @googleberg, modules you maintain (protobuf) have been updated in this PR. Please review the changes.

zhangskz
zhangskz previously approved these changes Jul 12, 2024
@Wyverald Wyverald added the presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval label Jul 12, 2024
bazel-io
bazel-io previously approved these changes Jul 12, 2024
Copy link
Member

@bazel-io bazel-io left a 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.

@fmeum
Copy link
Contributor

fmeum commented Jul 12, 2024

CI is failing

@bazel-io bazel-io dismissed stale reviews from zhangskz and themself July 16, 2024 16:42

Require module maintainers' approval for newly pushed changes.

zhangskz
zhangskz previously approved these changes Jul 16, 2024
bazel-io
bazel-io previously approved these changes Jul 16, 2024
Copy link
Member

@bazel-io bazel-io left a 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.

@fmeum
Copy link
Contributor

fmeum commented Jul 16, 2024

@zhangskz CI is still failing on Bazel 6

@zhangskz
Copy link
Contributor

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.
@bazel-io bazel-io dismissed stale reviews from zhangskz and themself July 17, 2024 15:48

Require module maintainers' approval for newly pushed changes.

Copy link
Member

@bazel-io bazel-io left a 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.

@Wyverald Wyverald merged commit fc73fd1 into bazelbuild:main Jul 17, 2024
22 checks passed
@zhangskz zhangskz deleted the protocolbuffers/[email protected] branch July 17, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants