-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Description
General overview of your idea.
The section Reset With a Path illustrates the use of the reset command in the forms including a <pathspec>. It reads well and the example is clear but there is this snippet about the syntax that I think might be slightly confusing:
This form (since you did not specify a commit SHA-1 or branch, and you didn't specify `--soft` or `--hard`) is shorthand for `git reset --mixed HEAD file.txt`...
Using --soft or --hard in this form would actually raise an error (which makes sense) but the current wording makes it sound like they could have been valid choices. I was surprised to see that --mixed is allowed (even if it triggers a deprecation warning) but I guess that this implementation detail is the reason why the git-reset documentation does not explicitly preclude the use of a <mode> alongside a <pathspec>.
I can see why elaborations around --mixed were dropped there for the sake of clarity, so I think the snippet above would also become clearer if mentions to <mode> were dropped altogether.
What problem will this solve?
The example in this section is good as it is, but some rephrasing would avoid potential confusion around mixing <mode> with <pathspec>, and also it would improve alignment between this section and the related documentation.
Have you thought about other solutions?
I'd say it’s quite clear that the git-reset documentation encourages the use of <mode> only in the form:
git reset [<mode>] [-q] [<commit>]
without elaborating on the caveat that --mixed could be used beyond this case.
In the same spirit, I think that the snippet above could benefit from being rephrased to something along the lines of:
This form (since you did not specify a commit SHA-1 or branch) is shorthand for `git reset HEAD -- file.txt`…
and perhaps adding a note with the caveat around --mixed if it's not too convoluted.
Would this be worth a pull request? I know a related issue was closed some years ago but in this case my point is not the warning when specifying --mixed.
Do you want to help with this enhancement idea?
Yes