Skip to content

Conversation

@rebello95
Copy link
Collaborator

Implements support for specifying additional options for protoc gRPC codegen to exclude client and/or server code.

Usage:

protoc TestEvent.proto --swiftgrpc_out=Client=true,Server=false:.

Also updated the readme with examples of all supported parameters.

@rebello95 rebello95 force-pushed the server-client-options branch from 0329885 to 5c5a79b Compare May 12, 2018 17:12
Implements support for specifying additional options for `protoc` gRPC codegen to exclude client and/or server code.

Usage:
```
protoc TestEvent.proto --swiftgrpc_out=Client=true,Server=false:.
```
@rebello95 rebello95 force-pushed the server-client-options branch from 5c5a79b to 7fc2a63 Compare May 12, 2018 17:23
@rebello95 rebello95 force-pushed the server-client-options branch from 7fc2a63 to ea1f7c8 Compare May 12, 2018 17:24
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.

LGTM

Very clean, thanks a lot!

@rebello95
Copy link
Collaborator Author

For sure! I had considered some alternatives - personally I think it'd be really nice to be able to do protoc MyProto.proto --swiftgrpc_out=. --outputs=client,server --visibility=internal but that's outside the realm of protoc's Google_Protobuf_Compiler_CodeGeneratorRequest as far as I know 🤷‍♂️

@timburks timburks merged commit 0375470 into grpc:master May 12, 2018
@rebello95 rebello95 deleted the server-client-options branch May 12, 2018 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants