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

Add --omit_package_doc flag #1702

Merged
merged 1 commit into from
Sep 28, 2020

Conversation

olivierlemasle
Copy link
Contributor

References to other Issues or PRs

Fixes #1701

Have you read the Contributing Guidelines?

Yes

Brief description of what is fixed or changed

Add an option -omit_package_doc to omit the package comment from generated code in gRPC gateway. This is useful if the generated code is added to an existing package that already has package documentation (the generated package comment might be misleading in that case).

Other comments

This is a rebased re-openening of #1103.

Makes it possible to omit the package comment from generated code. This
is useful if the generated code is added to an existing package that
already has package documentation (the generated package comment might
be misleading in that case).

Co-authored-by: olivierlemasle <[email protected]>
Copy link
Contributor

@srenatus srenatus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 😄 Thank you

Copy link
Collaborator

@johanbrandhorst johanbrandhorst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, thank you!

@johanbrandhorst
Copy link
Collaborator

Not sure how the Google CLA bot didn't react to this, but @RJPercival could you please give consent for this to be merged since it's based on your original PR?

@RJPercival
Copy link

It might be because I'm a xoogler. I'm happy for this to be merged 👍

@johanbrandhorst johanbrandhorst merged commit df2d592 into grpc-ecosystem:master Sep 28, 2020
@johanbrandhorst
Copy link
Collaborator

Thanks for your contribution! Could you please cherry-pick this against v2?

@olivierlemasle olivierlemasle deleted the pkgdoc branch September 28, 2020 15:53
olivierlemasle added a commit to olivierlemasle/grpc-gateway that referenced this pull request Sep 28, 2020
Makes it possible to omit the package comment from generated code. This
is useful if the generated code is added to an existing package that
already has package documentation (the generated package comment might
be misleading in that case).

Co-authored-by: olivierlemasle <[email protected]>

Co-authored-by: Rob Percival <[email protected]>
johanbrandhorst pushed a commit that referenced this pull request Sep 28, 2020
Makes it possible to omit the package comment from generated code. This
is useful if the generated code is added to an existing package that
already has package documentation (the generated package comment might
be misleading in that case).

Co-authored-by: olivierlemasle <[email protected]>

Co-authored-by: Rob Percival <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add option to remove (or remove) package comment in generated .pb.gw.go files
5 participants