rules_proto_grpc icon indicating copy to clipboard operation
rules_proto_grpc copied to clipboard

Fix grpclib_compile not working with paths with dashes

Open chrisirhc opened this issue 1 year ago • 0 comments

There was a missing |python conversion for grpclib. Modified the default thing example to demonstrate the failure in https://github.com/rules-proto-grpc/rules_proto_grpc/pull/320/commits/8685ee8671b261fe6fa3f236d6b80f25b3ddbbfb . However, I do see that a bunch of tests depend on the examples.proto path so I reverted it in my PR. Would like some opinions on whether I should make eamples.proto path contain a dash or leave the test change out of this PR. I'm concerned that this regression would occur again in the future.

I verified in https://github.com/vmagamedov/grpclib/blob/5916cba69fc1f8a25fe8e749e3c4427897b3d228/grpclib/plugin/main.py#L174 and https://github.com/vmagamedov/grpclib/blob/5916cba69fc1f8a25fe8e749e3c4427897b3d228/grpclib/plugin/main.py#L251 that |python is appropriate.

Fixes #321

chrisirhc avatar Jun 26 '24 03:06 chrisirhc