-
Notifications
You must be signed in to change notification settings - Fork 289
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get/Set repositories' default branch field #2326
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, just non-blocking question/comment.
@@ -55,6 +55,10 @@ message Repository { | |||
// url is the user configurable URL in the description of the repository, | |||
// always included the scheme and will not have a #fragment suffix. | |||
string url = 12; | |||
// default_branch in a BSR repository. It is used when syncing a git repository, to make sure both | |||
// default branches (BSR and Git) are in sync. By default, every BSR repository is created with a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens if the default branches (BSR and Git) are out of sync? Would it reject the sync until the user manually changes the default branch in BSR to match?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's correct, but that's in the buf sync
scope, in an upcoming PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's that PR: #2328
This allows
buf alpha repo sync
command to check if the local git default branch matches the BSR's, and abort if they don't.