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

Confusing wording and lack of explanation on Exploring History Episode #607

Open
eshook2010 opened this issue Apr 3, 2019 · 2 comments
Open

Comments

@eshook2010
Copy link

@eshook2010 eshook2010 commented Apr 3, 2019

The section discussing using git diff HEAD states "which is the same as what you would get if you leave out HEAD (try it)" with no explanation as to WHY it would be the same result, which is easy to infer but may not be clear to some learners. Also, the sentence before git diff HEAD~1 says "The real goodness in all this is when you can refer to previous commits" is a bit confusing and does not flow well with the rest of the lesson.

@billbrod
Copy link

@billbrod billbrod commented Aug 11, 2019

I agree that explaining it in more detail would be helpful. What about replacing "which is the same as what you would get if you leave out HEAD (try it)" with "HEAD is the default option for git diff, and so omitting it will not change the command's output at all (try it)."

I also agree that the "The real goodness in all this is when you can refer to previous commits" is a bit awkwardly phrased. How about something like "But the real power of git diff is the ability to compare far into the past, to compare with any previous commits. For example, we can look at the commit before HEAD by adding ~1 (where “~” is “tilde”, pronounced [til-duh])."

fmichonneau pushed a commit to fmichonneau/git-novice that referenced this issue Jul 1, 2021
@kekoziar
Copy link
Contributor

@kekoziar kekoziar commented Jul 31, 2021

The beginning of that episode says

You can refer to the most recent commit of the working directory by using the identifier HEAD.

Which sets up the next section (git diff HEAD is the same as git diff). Does that provide the clarification you're looking for?

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

Successfully merging a pull request may close this issue.

None yet
3 participants