Add static buf.yaml files for protovalidate and protovalidate-testing#694
Add static buf.yaml files for protovalidate and protovalidate-testing#694unmultimedio merged 3 commits intomainfrom
protovalidate and protovalidate-testing#694Conversation
|
The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).
|
|
What is the deal with this PR? |
|
Following up on this -- I recently put up bufbuild/protovalidate#366 to migrate to a v2 buf.yaml file, unaware of some of the obstacles. What is the status of being able to support it? @unmultimedio |
Sure we're ready if you wanna move to Lmk to update+merge this one first. |
Great. I'm ready if others think we're ok @rodaine @timostamm |
|
Think we're ready to when you are @unmultimedio thanks! |
| version: v1 | ||
| name: buf.build/bufbuild/protovalidate-testing | ||
| deps: | ||
| - buf.build/bufbuild/protovalidate |
There was a problem hiding this comment.
Not pinning to any specific version, so in case there are multiple breaking changes sequentially, a manual release of this repo needs to be done in between them to ensure synced remote dependencies don't break. See bufbuild/protovalidate#233 (comment)
unmultimedio
changed the title
protovalidate and protovalidate-testingprotovalidate and protovalidate-testing
@smaye81 this is merged! feel free to merge yours, and I'll ping you when the next release for both |
This is a small cleanup PR to convert the repo to use the v2 config for `buf.yaml`. It deletes the individual files in the proto directory in favor of a single one at the root which defines both modules. Also deleted `buf.work.yaml` and `buf.lock` since deps for local modules are handled by Buf. Note: should be merged _after_ bufbuild/modules#694
|
Just merged #366 @unmultimedio. Lmk if things look ok whenever. |
Sure, this repo will pick things up the next semver release. |
4 participants
Similar treatment than #579, waiting if/when protovalidate moves to v2: bufbuild/protovalidate#233