Skip to content

git reset does not affect commit, but changes index, stage to them#64

Closed
randomecho wants to merge 2 commits intogit:gh-pagesfrom
randomecho:63-correct-reset-notes
Closed

git reset does not affect commit, but changes index, stage to them#64
randomecho wants to merge 2 commits intogit:gh-pagesfrom
randomecho:63-correct-reset-notes

Conversation

@randomecho
Copy link
Copy Markdown
Contributor

Clear up what happens when you run git reset in that it does not affect the commit, or allow to change it per se, but that it affects what is sitting in the index and staging as pointed to by the commit or HEAD pointer mentioned.

This should make the notes actual now.

Clear up what happens when you run `git reset` in that it does
not affect the commit, or allow to change it per se, but that it
affects what is sitting in the index and staging as pointed to
by the commit or HEAD pointer mentioned.

Fixes git#63
@matthewmccullough
Copy link
Copy Markdown
Member

Looks like it nicely addresses the concerns in #63. Will want other @github/training-team members to check.

Comment thread basic/index.html Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Just a small modification to the description here?

"moves HEAD to specified commit reference, index and staging are untouched"

@randomecho
Copy link
Copy Markdown
Contributor Author

Thanks for the revision notes. Totally on board with the edits.

@adailey14
Copy link
Copy Markdown

Hi everyone, what's the story with this pull request? Was about to recommend this site as a great way to learn git basics but this change has to be made first. Happy to do any work needed to make this happen.

@randomecho
Copy link
Copy Markdown
Contributor Author

This looks merged.

@randomecho randomecho closed this Jan 10, 2014
@matthewmccullough
Copy link
Copy Markdown
Member

@randomecho @adailey14 Thank you both! Merged and shipped!

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.

4 participants