Conversation
Signed-off-by: Sri Krishna <skrishna@buf.build>
|
The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).
|
|
Tbh I'm not sure this will work, there is a check in the BSR backend that makes sure only the BSR syncer (and may be BSR admin, I will check) can push to modules marked as managed. Could you do a test on an ephemeral BSR first? I'll ping you internally with details. |
|
This feature is allowed and enabled in the public MT BSR now, please retry. One important note: this is allowed for all labels in both I'd recommend to disable manual pushes to |
We want |
Ok yes, in that case you'd want to keep
Correct, if someone depends on
Correct, but if/when
But I think that's ok for your needs? |
Yeah, we should be good. We can ask people to pin to a release version if depending on main is causing an issue. |
unmultimedio
left a comment
There was a problem hiding this comment.
Before merging, you might wanna update the branch, so it retries again and this PR branch is pushed?
https://buf.build/bufbuild/protovalidate/docs/sk/buf-push-ci |
I do not think we can push commits to the BSR's Let's say a new rule is added to We have to expect that users start using the new rule, but protovalidate implementations will not validate the new rule as expected: Most will raise an error for the unknown rule, but some will not (a silent correctness bug). Until we cut corresponding implementation releases, the only possible fix for the user is to pin to older version in buf.yaml. This problem already exists today, but only for the short window in time between protovalidate release and implementation releases. If we make this window bigger, the problem will become bigger. |
Good point. You could tweak this repo that if the Git's repo is |
This enables pushing to the BSR. Managed modules only push releases. This will make it so that every commit is available on the BSR. This helps with developing/testing features as they are added to protovalidate between releases. For example, the protovalidate-go uses generated SDKs and without the intermediate commits on BSR, it is impossible to get the updated gen code.