You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Looks like Methods use the helper protoSourceComments() to get comments, but it doesn't look like the Service comments are brought over into the generated sources, likely useful to still have them.
The text was updated successfully, but these errors were encountered:
Motivation:
It can be useful to know the service path a client is for, particularly
when constructing calls manually.
Additionally, we include the methods comments for RPCs but not the
service comments, they may be helpful in some cases.
Modifications:
- Generate the 'serviceName' for the client
- Generate the proto source comments for the client and server
Result:
- Generated code is a touch more useful
- Resolvesgrpc#962, grpc#1036
Motivation:
It can be useful to know the service path a client is for, particularly
when constructing calls manually.
Additionally, we include the methods comments for RPCs but not the
service comments, they may be helpful in some cases.
Modifications:
- Generate the 'serviceName' for the client
- Generate the proto source comments for the client and server
Result:
- Generated code is a touch more useful
- Resolves#962, #1036
Looks like Methods use the helper
protoSourceComments()
to get comments, but it doesn't look like the Service comments are brought over into the generated sources, likely useful to still have them.The text was updated successfully, but these errors were encountered: