Skip to content

Minor additions #2

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

Closed
rorysedgwick opened this issue Mar 31, 2015 · 3 comments
Closed

Minor additions #2

rorysedgwick opened this issue Mar 31, 2015 · 3 comments
Assignees

Comments

@rorysedgwick
Copy link

Possible inclusions based on what I found confusing when learning version control:

On the line about adding comments to commits: emphasise the necessity of a message otherwise you end up in VIM, which is pretty confusing and difficult to exit if you haven't seen it before.

Explain that when using push or pull, 'origin' refers to the location(usually/always? the remote repo) and 'master' is referring to the branch there. For a while I did not understand the difference between them.

That merging locally and merging on GitHub is the same step twice, so not mandatory but gives extra control of remote repo.

When pushing to the remote repo the new branch has to have the exact same name as the local branch as you are pushing a copy of it.

A couple of times referring to a pull request as a PR may be confusing for those not familiar with the overall process.

These feel like very nitpicky points for what will be an amazing learning resource for newcomers to Git, but are things I was slightly confused by to begin with so may be worth including.

@NataliaLKB
Copy link
Owner

👍 great feedback. Exactly what this tutorial needs. I will put in some though and let you know when I have updated the tutorial and you can let me know if it is better 😄

@NataliaLKB
Copy link
Owner

@rorysedgwick please check my PR #3 . Let me know if I have clarified things a bit better! Thanks.

@NataliaLKB NataliaLKB self-assigned this Apr 1, 2015
@rorysedgwick
Copy link
Author

Yep that seems clearer and covers all the bits I thought were missing

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

No branches or pull requests

2 participants