bufbuild
Add buf convert --disallow-unknown
Begins https://github.com/bufbuild/buf/issues/2714. See https://github.com/bufbuild/protoyaml-go/issues/37 for the remaining work for YAML that once fixed, can resolve the aforementioned issue.
The latest Buf updates on your PR.
| Name | Status |
|---|---|
| build | ✅ passed |
| lint | ✅ passed |
| format | ✅ passed |
| breaking | ✅ passed |
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
This was just a drive-by PR and it's actually wrong. This adjusts the default behavior for binary and text formats to discard unknown values, which breaks a lot of things in unexpected ways. For some reason, we had discard unknown default to true for JSON, false for wire and txtpb, and implicitly false for yaml. This was likely a decision I made a long time ago.
This PR will need to be significantly adjusted to not introduce breaking changes.