Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
feat(bazel): detach spring bazel rules #1065
feat(bazel): detach spring bazel rules #1065
Changes from all commits
5bd637e
d67df67
5327150
abb931d
6df1644
767fee9
0ba426a
2310c81
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
I suppose this is derived from
_java_gapic_library()
injava_gapic.bzl
and you removed a couple of options. Can you double check if they don't affect sping-codegen for us?E.g. I noticed
transport
is removed, If I get it right, this is where transport is specified asgrpc
orgrpc+rest
orrest
, which get parsed in and eventually plays a part in #1078. Are we losing this info here?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.
Oh, we indeed need
transport
to be specified! I'll double check the impact of the other options.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.
So, the other option is
rest_numeric_enums
, which is used byHttpJsonServiceStubClassComposer
to determine how to serialize enums (by number or value). I see that our spring composers do not mention enums, so it should be safe to remove.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.
It is probably not in scope now, but once SpringCodeGen decide to support both
grpc
andrest
, we need a way to pass the correct value to this Spring specific bazel rule, and currently the only source of truth is the Bazel files in googleapis.