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

bzlmod: Remove build_file_proto_mode attribute #3506

Merged
merged 1 commit into from
Mar 31, 2023

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Mar 31, 2023

What type of PR is this?

Other

What does this PR do? Why is it needed?

By updating gazelle to a version that maintains a central registry of gazelle overrides, specifying build_file_proto_mode on github.com/gogo/protobuf is no longer necessary.

This will allow us to get rid of the rules_go exception after the next rules_go release.

Which issues(s) does this PR fix?

Fixes #

Other notes for review

By updating gazelle to a version that maintains a central registry of
gazelle overrides, specifying `build_file_proto_mode` on
`github.com/gogo/protobuf` is no longer necessary.

This will allow us to get rid of the rules_go exception after the next
rules_go release.
@fmeum
Copy link
Collaborator Author

fmeum commented Mar 31, 2023

@tyler-french

@fmeum fmeum merged commit e558f56 into bazelbuild:master Mar 31, 2023
1 check passed
linzhp pushed a commit that referenced this pull request Apr 20, 2023
By updating gazelle to a version that maintains a central registry of
gazelle overrides, specifying `build_file_proto_mode` on
`github.com/gogo/protobuf` is no longer necessary.

This will allow us to get rid of the rules_go exception after the next
rules_go release.
tingilee pushed a commit to tingilee/rules_go that referenced this pull request Jul 19, 2023
By updating gazelle to a version that maintains a central registry of
gazelle overrides, specifying `build_file_proto_mode` on
`github.com/gogo/protobuf` is no longer necessary.

This will allow us to get rid of the rules_go exception after the next
rules_go release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants