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
Use case(s) - what problem will this feature solve?
We can add comments to a service in our proto file.
// Here we can document how the service works, when to use it, etc.serviceMyService {
}
However, they are currently completely ignored by protoc-gen-go-grpc. It instead puts a boilerplate comment on the client and server interfaces.
// MyServiceClient is the client API for MyService service.//// For semantics around ctx use and closing/ending streaming RPCs, please refer to https://pkg.go.dev/google.golang.org/grpc/?tab=doc#ClientConn.NewStream.typeMyServiceClientinterface {
}
// MyServiceServer is the server API for MyService service.// All implementations must embed UnimplementedMyServiceServer// for forward compatibilitytypeMyServiceServerinterface {
}
Proposed Solution
Fix protoc-gen-go-grpc to copy over the comments from the service to the interfaces instead of the boilerplate comments.
Note that generators for other languages already do this properly.
Alternatives Considered
None.
Additional Context
None.
The text was updated successfully, but these errors were encountered:
This is a good to have feature but currently considered lower priority for the team. We would be happy to review if you can send a PR for this. Thanks.
@purnesh42H Hello, This Bug, i was fix in my latest commit, already create PR, if you have any question, please point out.I will modify it immediately, Thanks!
Use case(s) - what problem will this feature solve?
We can add comments to a service in our proto file.
However, they are currently completely ignored by protoc-gen-go-grpc. It instead puts a boilerplate comment on the client and server interfaces.
Proposed Solution
Fix protoc-gen-go-grpc to copy over the comments from the service to the interfaces instead of the boilerplate comments.
Note that generators for other languages already do this properly.
Alternatives Considered
None.
Additional Context
None.
The text was updated successfully, but these errors were encountered: