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

contributing: commiter git FAQ #68

Closed
wants to merge 2 commits into from
Closed

Conversation

indutny
Copy link
Member

@indutny indutny commented Dec 4, 2014

Describe in details how our current git flow works and could be used.

Fix #67

Describe in details how our current git flow works and could be used.
Fix nodejs#67

```sh
git fetch origin
git merge --ff-only origin/v0.12
Copy link
Member

Choose a reason for hiding this comment

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

Is there a reason to prefer fetch + ff-only merge over git pull --rebase?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think the fetch + merge is much more explicit.

Copy link
Contributor

Choose a reason for hiding this comment

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

+1 on more explicit

@bnoordhuis
Copy link
Member

Left two suggestions and there is a typo in the commit log (s/commiter/committer/) but, apart from that, LGTM.

@Fishrock123
Copy link
Contributor

+1

@indutny
Copy link
Member Author

indutny commented Dec 4, 2014

@bnoordhuis pushed a fix

@bnoordhuis
Copy link
Member

Cheers, LGTM.

indutny added a commit that referenced this pull request Dec 4, 2014
Describe in details how our current git flow works and could be used.

Fix #67
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
PR-URL: #68
@indutny
Copy link
Member Author

indutny commented Dec 4, 2014

Landed in dca1ee4, thank you!

@indutny indutny closed this Dec 4, 2014
@indutny indutny deleted the fix/gh-io-67 branch December 4, 2014 15:20
@mikeal
Copy link
Contributor

mikeal commented Dec 4, 2014

wasn't there talk of automating all of this?

@indutny
Copy link
Member Author

indutny commented Dec 4, 2014

@mikeal there was, but knowing how to use git is still good. And this could be a good starting point for a people working on that tool.

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.

5 participants