Hmmm... You might be right. I've implemented wider logic than needed. On the other hand fixing rules like that, when they already exist anyway, may be useful. I mean, e.g. setting proto_only = True when no gRPC services exist in the .proto file, etc.
So what do you think about moving some part of this logic to the Fix() method? This way, we still encourage the use of the better style, but if somebody already has "deprecated" rules, we can at least help with checking that everything is correct.
Originally posted by @mateusz-olczyk in #194 (comment)
Hmmm... You might be right. I've implemented wider logic than needed. On the other hand fixing rules like that, when they already exist anyway, may be useful. I mean, e.g. setting
proto_only = Truewhen no gRPC services exist in the.protofile, etc.So what do you think about moving some part of this logic to the
Fix()method? This way, we still encourage the use of the better style, but if somebody already has "deprecated" rules, we can at least help with checking that everything is correct.Originally posted by @mateusz-olczyk in #194 (comment)