From: D. Ben Knoble Date: Fri, 09 Jan 2026 23:24:19 GMT Subject: Re: [PATCH v8 7/7] builtin/history: implement "reword" subcommand Message-ID: In-Reply-To: [I think this is present in v9 or I wouldn't bother following up :)] On Fri, Jan 9, 2026 at 2:37 AM Patrick Steinhardt wrote: > > On Wed, Jan 07, 2026 at 01:01:11PM -0500, D. Ben Knoble wrote: > > On Wed, Jan 7, 2026 at 5:10 AM Patrick Steinhardt wrote: > > > > > > Implement a new "reword" subcommand for git-history(1). This subcommand > > > is similar to the user performing an interactive rebase with a single > > > commit changed to use the "reword" instruction. > > > > > diff --git a/Documentation/git-history.adoc b/Documentation/git-history.adoc > > > index 5a9d931efc..4eea317e5c 100644 > > > --- a/Documentation/git-history.adoc > > > +++ b/Documentation/git-history.adoc > > > @@ -32,8 +32,9 @@ Overall, linkgit:git-history[1] aims to provide a more opinionated way to modify > > > your commit history that is simpler to use compared to linkgit:git-rebase[1] in > > > general. > > > > > > -If you want to reapply a range of commits onto a different base, or interactive > > > -rebases if you want to edit a range of commits. > > > +Use linkgit:git-rebase[1] if you want to reapply a range of commits onto a > > > +different base, or interactive rebases if you want to edit a range of commits > > > +at once. > > > > > > LIMITATIONS > > > ----------- > > > > Ah, looks like this commit fixed one of my befuddlements during the range-diff. > > Bad history edit? :) > > Huh, something's weird here indeed, the range-diff and the actual > changes do not match. I use b4 though, so this isn't an issue that was > caused by weird manual munging of patches. Hum. > > Well, I'll not go down this rabbit hole today :) Thanks for your review! > > Patrick What's weird is the pre-image, which comes from the patch right before it. It seems like an incomplete sentence that could be dropped from what I see in Junio's mirror as baf46a880d (builtin: add new "history" command, 2026-01-09). -- D. Ben Knoble