On Tue, Nov 21, 2017 at 04:32:42PM -0800, Stefan Beller wrote: > > Heh, yes. I even fixed it once, but I have the funny habit of noticing > > such typos while reading the "todo" list of "rebase -i" and fixing them > > there. Which of course has no impact whatsoever on the commit. :-/ > > That happened to me a couple of times as well before. > This sounds like a UX bug on first thought. > > On second thought the text after the abbreviated hash can be > user dependent IIRC, by setting some format option how to populate the > rebase instruction sheet using log, so it would not be easy to take > fixes from that line into the commit for a fixup. Right, I came to the same conclusion (we may even have discussed this on the list, I don't remember). The current "todo" format says that only the command and sha1 matter, and we'd be changing that. Maybe that's not so bad if the user has to enable the feature themselves (and clearly it would be incompatible with a custom format option). But I think in the end it probably just makes sense to retrain my expectation, and remember to "reword" instead of "pick". -Peff
