Skip to content

Conversation

@glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Mar 11, 2020

Motivation:

We have access to the underlying comments associated with RPCs in the
protobuf definition. These often explain what each RPC does and can have
great value to the user.

Modifications:

  • Generate documentation for client stubs and server provider protocol
    based on the source comments in the proto file.
  • For clients: fall back to some basic documentation when no comments
    are present
  • Re-generate code.

Result:

  • The generate code has more helpful documentation

Motivation:

We have access to the underlying comments associated with RPCs in the
protobuf definition. These often explain what each RPC does and can have
great value to the user.

Modifications:

- Generate documentation for client stubs and server provider protocol
  based on the source comments in the proto file.
- Re-generate code.

Result:

- The generate code has more helpful documentation
@glbrntt glbrntt added nio 🔨 semver/patch No public API change. labels Mar 11, 2020
@glbrntt
Copy link
Collaborator Author

glbrntt commented Mar 11, 2020

Resolves #597

@glbrntt glbrntt requested a review from MrMage March 11, 2020 15:52
Copy link
Collaborator

@MrMage MrMage left a comment

Choose a reason for hiding this comment

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

Nice!

@glbrntt glbrntt merged commit 522e5d4 into grpc:nio Mar 13, 2020
@glbrntt glbrntt deleted the gb-generate-proto-documentation branch August 5, 2020 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 semver/patch No public API change.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants