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

command for merging updates from base branch #74

Closed
rarkins opened this issue Nov 12, 2014 · 4 comments
Closed

command for merging updates from base branch #74

rarkins opened this issue Nov 12, 2014 · 4 comments

Comments

@rarkins
Copy link

rarkins commented Nov 12, 2014

As a policy, we ask developers to merge updates from the base branch (e.g. master) prior to submitting or resubmitting a PR for review. Due to merge squashing, it won't matter how many messy merge commits come, so this is good practice for a project where people are committing things that don't cause merge conflict but could still cause problems. A reflow command for this would be good.

@codenamev
Copy link
Collaborator

That's a great idea. I think this should be built into the review command. @nhance anything against pulling up master during review?

@rarkins
Copy link
Author

rarkins commented Dec 3, 2014

@codenamev It would be good to have it as a part of review but also accessible on its own. i.e. a developer knows that a major update was made to mainline, it will break his branch, so he needs to merge and then fix before continuing.

@codenamev
Copy link
Collaborator

simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 18, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 18, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 18, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 18, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 18, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 18, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 18, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 18, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 19, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 19, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 20, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 20, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 20, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 20, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 20, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 20, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 20, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 20, 2016
simonzhu24 added a commit to simonzhu24/gitreflow that referenced this issue Apr 21, 2016
codenamev pushed a commit that referenced this issue Apr 21, 2016
@codenamev
Copy link
Collaborator

Fixed in v0.8.0 with 3b44510. git reflow refresh 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants