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

Re: How to commit incomplete changes?

From
Alexey Shumkin <alex.crezoff@gmail.com>
Date
Dec 15, 2011, 08:22 UTC
Message-ID
<20111215122252.584d1003@ashu.dyn1.rarus.ru>
In-Reply-To
<7e1ccfac8c47e8877c0438086bd1d91b@ulrik.uio.no>

Oh! I got it. I missed "generated changes". Well, unfortunately (or fortunately ;) ), I did not meet such a workflow when changes are "generated" without my hands. In your case it may sound reasonable to make separate fixup commit. But Git allows you to make your own (more flexible than SVN, for instance) workflow, which suits you. It's up to you ;) You decide. If you plan cherry-picking that fixups, do separate fixups. Just publish them together. If you want every commit is "clear" and "workable", squash fixup into a single commit. I do not know what exactly is "generated changes" you're talking about ), so, maybe I'd do separate fixups, maybe not. )) There is no single solution. ))) TIMTOWTDI That is why you hesitate :) Do your own decision. And feel free to change it later. ))

Show 13 quoted lines
>  To expand on that: To review the change, review the hand-edited 
>  commits,
>  which is easier when these do not drown in generated changes.  Review
>  the *commands* which generated the rest - I'd put those in the commit
>  message - and glance at the actual changes.  Cherry-pick: Possbly you
>  need to run the commands instead of cherry-picking the generated
>  changes.  That's easier with a commit with only generated changes.
> 
>  I know it also can cause problems.  Would you make a single big
> commit anyway, and describe carefully in the commit message which
> parts are hand-edits?  (We don't auto-test commits yet, but I'll sure
> this issue will crop up again later when we do.)
> 
Previous: Hallvard B FurusethNext: Alexey Shumkin
Message 4 of 10 in “How to commit incomplete changes?”
  1. Hallvard B FurusethDec 14, 2011
  2. Alexey ShumkinDec 15, 2011
  3. Hallvard B FurusethDec 15, 2011
  4. Alexey ShumkinDec 15, 2011
  5. Alexey ShumkinDec 15, 2011
  6. Neal KreitzingerDec 15, 2011
  7. Junio C HamanoDec 16, 2011
  8. Hallvard Breien FurusethDec 16, 2011
  9. Hallvard Breien FurusethDec 16, 2011
  10. Tomas CarneckyDec 16, 2011

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.