git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Stage, test, and commit only some changes, then repeat

From
Sergey Organov <sorganov@gmail.com>
Date
Jan 31, 2022, 21:56 UTC
Message-ID
<87iltzpo1r.fsf@osv.gnss.ru>
In-Reply-To
<49031AA3-CDC3-4CF6-A04B-E3533907FCA4@gmail.com>
Géry Ogam <gery.ogam@gmail.com> writes:
Show 63 quoted lines
>> Le 31 janv. 2022 à 17:27, Sergey Organov <sorganov@gmail.com> a écrit :
>> 
>> Géry Ogam <gery.ogam@gmail.com> writes:
>> 
>>> Hello,
>>> 
>>> I would like to stage, test, and commit only *some* changes of the
>>> working tree, and then repeat this process with the remaining changes.
>>> 
>>> My current solution (published at
>>> https://stackoverflow.com/a/70914962/2326961):
>>> 
>>> 1. Stage some changes:
>>>   ```
>>>   git add -p file
>>>   ```
>>> 2. Save away the remaining changes:
>>>   ```
>>>   git diff >patch
>>>   git stash push -k
>>>   ```
>>> 3. Test the staged changes.
>>> 4. Commit the staged changes:
>>>   ```
>>>   git commit
>>>   ```
>>> 5. Restore the remaining changes:
>>>   ```
>>>   git apply patch
>>>   ```
>>> 6. Go to step 1.
>>> 
>>> It is not ideal because a) it uses a patch file for saving the
>>> remaining changes; b) it uses the stash only for setting the working
>>> tree to the index state.
>>> 
>>> It would be ideal if I could save *only* the remaining changes in the
>>> stash instead of resorting to a patch file. How to do it?
>> 
>> It looks like you don't need patch file for this workflow. What's wrong with:
>> 
>> git add...
>> git stash push --keep-index
>> ... check, git add fixes
>> git commit
>> git stash apply
>> 
>> ???
>> 
>> -- Sergey Organov
>
> Hello Sergey,
>
> `git stash` saves the transition from the HEAD state to the working
> tree state. It also sets the working tree to the *HEAD* state.
>
> `git stash --keep-index` saves the transition from the HEAD state to
> the working tree state. It also sets the working tree to the *index*
> state.
>
> `git stash pop` applies the last saved transition. So if the working
> tree was not in HEAD state (like after `git stash --keep-index`),
> there will be a conflict.

Did you actually try it and got conflict? I doubt there will be any if you don't modify anything after "git stash --keep-index" during testing, and if you do, than any method might bring conflicts.

In fact I just re-tested this to make sure, and got no conflicts.
-- Sergey Organov
Previous: Géry OgamNext: Géry Ogam
Message 6 of 8 in “Stage, test, and commit only some changes, then repeat”
  1. Géry OgamJan 30, 2022
  2. Johannes SixtJan 30, 2022
  3. Géry OgamJan 31, 2022
  4. Sergey OrganovJan 31, 2022
  5. Géry OgamJan 31, 2022
  6. Sergey OrganovJan 31, 2022
  7. Géry OgamJan 31, 2022
  8. Sergey OrganovFeb 1, 2022

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.