-
Notifications
You must be signed in to change notification settings - Fork 655
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
add importpath_aliases to go_proto_library #2608
Conversation
This PR seems incomplete? It doesn't actually implement |
@jayconrod in Line 19 of |
Im pretty new to rules_go but i think it works out of the box because |
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.
Oops, I'm sorry, you're totally right. I forgot how this worked and was thinking that adding this attribute would be a lot more complicated.
This PR looks good, just have two small comments. Thanks for fixing this.
np! @jayconrod responded to your comments! |
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.
Looks good, thanks!
This reverts commit 631e26b. rules_go CI started failing after bazelbuild#2608 was merged. For bazelbuild/bazel#11885
What type of PR is this?
Bug fix
What does this PR do? Why is it needed?
It allows go_proto_library to accept
importpath_aliases
as an argument. It is needed because gazelle generates go_proto_library rules that have importpath_aliases as an argument.Which issues(s) does this PR fix?
Fixes #2358