bufdev
bufdev
Yea I wasn't thinking clearly, definitely true re: local fs paths (and the bane of my existence). Some other options: - Straight up exclude the WKTs for dir=prefix. This might...
Sorry, maybe I wasn’t clear - I’m not proposing ditching protoreflect. You’d still use protoreflect, but instead of basing your lint rules on the types in protoreflect, you’d use a...
@lukesneeringer for context...specifically what I'm approximately suggesting here: - Use a pure interface to represent all the descriptor types, and not rely on the protoreflect structs. Buf does this with...
We created all the wrappers already at https://github.com/bufbuild/buf/tree/master/internal/pkg/protodesc that represent the full `FileDescriptorSet`. It was somewhat time-consuming, but the work is done. Yes, they do support dependencies/imports as necessary for...
I'd really recommend checking out https://godoc.org/github.com/bufbuild/buf/internal/pkg/protodesc
It's definitely not another protoreflect library, it just exposes interfaces suitable for lint rules that are as minimal as possible. I'd be able to put up a PR that migrates...
I can also confirm that `make ci` does not work in Docker out of the box on the current commit on master: ``` $ git log -n 1 commit 76a978908cf54afdb586f9725274a67ede4b99af...
Closing this for now - it's an interesting idea, but I do not see us doing this anytime in the near future.
Closing this as wontfix for now - this would imply that `buf breaking` needs to perform a type of checking that is non-transitive. We always want to make sure that...
Tracking on https://github.com/bufbuild/buf/issues/29