Skip to content
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

Sync single branch by default #2329

Merged
merged 23 commits into from
Aug 1, 2023
Merged

Conversation

unmultimedio
Copy link
Member

@unmultimedio unmultimedio commented Jul 31, 2023

The expected most common use case for the sync command is in CI, on pull requests or individual pushes to remotes. This means we should sync a single branch by default, whatever one I'm currently checked in. Also added a flag --all-branches to go in the usual sync order and sync all remote branches.

Base automatically changed from jfigueroa/make-sure-default-branches-are-in-sync to main August 1, 2023 16:42
Copy link
Member

@doriable doriable left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look reasonable, just a nit/question.

Copy link
Member

@doriable doriable left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@unmultimedio unmultimedio merged commit d5d7c0d into main Aug 1, 2023
6 checks passed
@unmultimedio unmultimedio deleted the jfigueroa/sync-single-branch branch August 1, 2023 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants