Re: [PATCH v4 3/6] commit: add a reword suboption to --fixup
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Mar 11, 2021, 06:37 UTC
- Message-ID
- <CAPig+cQYx+mqMSe=+7RFh0Ft3Sx110n+QzDKMZaOGJoc+R0H5w@mail.gmail.com>
- In-Reply-To
- <xmqqa6rafdyg.fsf@gitster.g>
On Thu, Mar 11, 2021 at 12:37 AM Junio C Hamano <gitster@pobox.com> wrote:
Show 16 quoted lines
> My suggestion primarily started a reaction to that "As `amend`..." > which was not gramatically complete sentence, and I ended up > rewriting everything after "As `amend`..." > > But re-reading what is in the paragraph before, I tend to think that > it places too much stress on 'colon' and should be removed. > > The comment is about what is being parsed, so > > We limit --fixup's suboptions to only alpha characters. If > the first character after a run of alpha is colon, then the > part before the colon may be a known suboption name like > `amend` or `reword`, or a misspelt suboption name. In > either case, we treat it as --fixup=<suboption>:<arg> > > Otherwise, we are dealing with --fixup=<commit>.
I suggested a different way to rewrite this comment[1] in my review of patch [2/6] before I was aware that you two were discussing rewrites here in [3/6].
[1]: https://lore.kernel.org/git/CAPig+cRxZFV7DDtnYOxRqEYU4PwXK8gb2HSehDyGorqdgsmoNQ@mail.gmail.com/