Re: [PATCH v3 3/4] doc: git-reset: clarify `git reset [mode]`
On Thursday, 1 January 2026 23:43:58 CET D. Ben Knoble wrote:
Show 46 quoted lines
> From: Julia Evans <julia@jvns.ca>
>
> From user feedback, there was some confusion about the differences
> between the modes, including:
>
> 1. Sometimes it says "index" and sometimes "index file".
> Fix by replacing "index file" with "index".
> 2. Many comments about not being able to understand what `--merge` does.
> Fix by mentioning obscure situations, since that seems to be what
> it's for. Most folks will use `git <cmd> --abort`.
> 3. Issues telling the difference between --soft and --mixed, as well as
> --keep. Leave --keep alone because I couldn't understand its use case,
> but change `--soft` / `--mixed` / `--hard` as follows:
>
> --mixed is the default, so put it first.
>
> Describe --soft/--mixed/--hard with the following structure:
>
> * Start by saying what happens to the files in the working directory,
> because the thing users want to avoid most is irretrievably losing
> changes to their working directory files.
> * Then describe what happens to the staging area. Right now it seems to
> frame leaving the index alone as being a sort of neutral action.
> I think this is part of what's confusing users, because in Git when
> you update HEAD, Git almost always updates the index to match HEAD.
> So leaving the index unchanged while updating HEAD is actually quite
> unusual, and it deserves to be flagged.
> * Finally, give an example for --soft to explain a common use case.
>
> Signed-off-by: Julia Evans <julia@jvns.ca>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
> Signed-off-by: D. Ben Knoble <ben.knoble+github@gmail.com>
> ---
> Documentation/git-reset.adoc | 44 ++++++++++++++++++++----------------
> 1 file changed, 24 insertions(+), 20 deletions(-)
>
> diff --git a/Documentation/git-reset.adoc b/Documentation/git-reset.adoc
> index 71e8f52430..6de0d524c3 100644
> --- a/Documentation/git-reset.adoc
> +++ b/Documentation/git-reset.adoc
> @@ -24,42 +24,46 @@ DESCRIPTION
> the staged version of the specified files.
>
> `git reset [<mode>] [<commit>]`::
> - This form resets the current branch head to _<commit>_ and
> - possibly updates the index (resetting it to the tree of _<commit>_)
> - the working tree depending on _<mode>_. Before the operation,
> - is set to the tip of the current branch. If _<mode>_ is omitted,
> - defaults to `--mixed`. The _<mode>_ must be one of the following:
> + Set the current branch head (`HEAD`) to point at _<commit>_.
> + Depending on _<mode>_, also update the working directory and/or
> + to match the contents of _<commit>_.
> + _<commit>_ defaults to `HEAD`.
> + Before the operation, `ORIG_HEAD` is set to the tip of the current
Show 17 quoted lines
> ++
> +The _<mode>_ must be one of the following (default `--mixed`):
> +
> ---
> -`--soft`::
> - Does not touch the index file or the working tree at all (but
> - resets the head to _<commit>_, just like all modes do). This leaves
> - all your changed files "Changes to be committed", as `git status`
> - would put it.
>
> +--
> `--mixed`::
> - Resets the index but not the working tree (i.e., the changed files
> - are preserved but not marked for commit) and reports what has not
> - been updated. This is the default action.
> + Leaves your working directory unchanged.
> + Updates the index to match the new HEAD, so nothing will be staged.
Please use imperative mood here, and use `HEAD`.
Show 10 quoted lines
> +
> If `-N` is specified, removed paths are marked as intent-to-add (see
> linkgit:git-add[1]).
>
> +`--soft`::
> + Leave your working tree files and the index unchanged.
> + For example, if you have no staged changes, you can use
> + `git reset --soft HEAD~5; git commit`
> + to combine the last 5 commits into 1 commit. This works even with
> + changes in the working tree, which are left untouched, but such
> + can lead to confusion.
> +
> `--hard`::
> - Resets the index and working tree. Any changes to tracked files in
> - working tree since _<commit>_ are discarded. Any untracked files or
> - directories in the way of writing any tracked files are simply
> + Overwrites all files and directories with the version from
> + and may overwrite untracked files. Tracked files not in _<commit>_
> + removed so that the working tree matches _<commit>_.
> + Updates the index to match the new HEAD, so nothing will be staged.
>
> `--merge`::
> + Mainly exists to reset unmerged index entries, like those left
> + `git am -3` or `git switch -m` in certain situations.
Personal preference, please insert this context sentence after the main description of the action. Could you also change the mood of the description to imperative?
> Resets the index and updates the files in the working tree that are
> different between _<commit>_ and `HEAD`, but keeps those which are
> different between the index and working tree (i.e. which have
Show 5 quoted lines
> which have not been added).
> If a file that is different between _<commit>_ and the index has
> unstaged changes, reset is aborted.
> -+
> -In other words, `--merge` does something like a `git read-tree -u -m
> -but carries forward unmerged index entries.
>
> `--keep`::
> Resets index entries and updates files in the working tree that are