From: Dennis Kaarsemaker Date: Wed, 31 Aug 2016 18:39:31 GMT Subject: Re: [PATCH 06/34] sequencer (rebase -i): write the 'done' file Message-ID: <1472668771.4265.74.camel@kaarsemaker.net> In-Reply-To: <516d2aaf8837c8675141cf9c330c18c44765b69d.1472633606.git.johannes.schindelin@gmx.de> On wo, 2016-08-31 at 10:54 +0200, Johannes Schindelin wrote: > In the interactive rebase, commands that were successfully processed are > not simply discarded, but appended to the 'done' file instead. This is > used e.g. to display the current state to the user in the output of > `git status` or the progress. Wouldn't it make more sense to have this patch before the ones that implement the actual rebase commands? Hmm, and after reading more of this series, I think the same applies to some other patches too, e.g. 08/34 and 14/34, so I'm probably missing something. So before I make a fool of myself and suggest that the implementation of the actual commands should come at the end, maybe you could tell me what I'm missing :)  D.