Re: [PATCH v1 11/11] doc: promote "git restore"
From: Elijah Newren <hidden>
Date: 2019-03-09 19:37:46
On Fri, Mar 8, 2019 at 2:17 AM Nguyễn Thái Ngọc Duy [off-list ref] wrote:
The new command "git restore" (together with "git switch") are added to avoid the confusion of one-command-do-all "git checkout" for new users. They are also helpful to avoid ambiguation context.
s/ambiguation/ambiguous/ or s/ambiguation context/ambiguity/?
For these reasons, promote it everywhere possible. This includes documentation, suggestions/advice from other commands. One nice thing about git-restore is the ability to restore "everything", so it can be used in "git status" advice instead of both "git checkout" and "git reset". The three commands suggested by "git status" are add, rm and restore. "git checkout" is also removed from "git help" (i.e. it's no longer considered a commonly used command) Signed-off-by: Nguyễn Thái Ngọc Duy <redacted> ---
quoted hunk ↗ jump to hunk
@@ -56,7 +56,7 @@ summary of what is included by any of the above for the next commit by giving the same set of parameters (options and paths). If you make a commit and then find a mistake immediately after -that, you can recover from it with 'git reset'. +that, you can recover from it with 'git restore' or 'git reset'.
Does 'git revert' also belong in this set? And do we need to extend our blurb differentiating reset and restore to also include revert?
quoted hunk ↗ jump to hunk
@@ -26,8 +26,8 @@ effect of some earlier commits (often only a faulty one). If you want to throw away all uncommitted changes in your working directory, you should see linkgit:git-reset[1], particularly the `--hard` option. If you want to extract specific files as they were in another commit, you -should see linkgit:git-checkout[1], specifically the `git checkout -<commit> -- <filename>` syntax. Take care with these alternatives as +should see linkgit:git-restore[1], specifically the `--source` +option Take care with these alternatives as both will discard uncommitted changes in your working directory.
Missing period after "option".
------------------------------------------------- -$ git reset --hard HEAD +$ git merge --abort
Nice additional cleanup, but should it be in a separate patch since it isn't about the restore command? Elijah