All of lore.kernel.org
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Charvi Mendiratta <charvi077@gmail.com>
Cc: git <git@vger.kernel.org>,
	Christian Couder <christian.couder@gmail.com>,
	Phillip Wood <phillip.wood123@gmail.com>
Subject: Re: What's cooking in git.git (Feb 2021, #03; Wed, 17)
Date: Thu, 18 Feb 2021 11:36:57 -0800	[thread overview]
Message-ID: <xmqq8s7lf9ra.fsf@gitster.g> (raw)
In-Reply-To: <CAPSFM5efu6MF2RzfDkw8XJ9kF79akEesdA0WFAG3n9fo9N26vQ@mail.gmail.com> (Charvi Mendiratta's message of "Thu, 18 Feb 2021 16:14:23 +0530")

Charvi Mendiratta <charvi077@gmail.com> writes:

> On Thu, 18 Feb 2021 at 07:44, Junio C Hamano <gitster@pobox.com> wrote:
>>
>> * cm/rebase-i-fixup-amend-reword (2021-02-17) 6 commits
>>  - doc/git-commit: add documentation for fixup[amend|reword] options
>>  - t3437: use --fixup with options to create amend! commit
>>  - t7500: add tests for --fixup[amend|reword] options
>>  - commit: add a reword suboption to --fixup
>>  - commit: add amend suboption to --fixup to create amend! commit
>>  - sequencer: export subject_length()
>>  (this branch uses cm/rebase-i and cm/rebase-i-updates.)
>>
>>  "git commit --fixup" learns to optionally create "amend!" and
>>  "reword!"  commits, that use the log message in "fixup" commit
>>  as the message of the resulting commit.
>>
>
> I would like to put light on the description of the patches. I might have
> misinterpreted the meaning of  _"reword!" commit_ as written above.
> But I thought to explain that here both additional suboptions i.e
> `amend` and `reword`
> of `--fixup` introduced in the patch series creates "amend!" commit only.

Thanks for clarifying.  I did mix up in the description.

As the parameter given to the command line option is much more
end-user facing than which insn it results in the todo list, how
about explaining the topic like so:

    "git commit --fixup=<commit>", which was to tweak the changes
    made to the contents while keeping the original log message
    intact, learned "--fixup=(amend|reword):<commit>", that can be
    used to tweak both the message and the contents, and only the
    message, respectively.


  reply	other threads:[~2021-02-18 19:50 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-18  2:11 What's cooking in git.git (Feb 2021, #03; Wed, 17) Junio C Hamano
2021-02-18  3:19 ` Matheus Tavares Bernardino
2021-02-18 18:25   ` Junio C Hamano
2021-02-18  6:44 ` Martin Ågren
2021-02-18 19:31   ` Junio C Hamano
2021-02-18 10:44 ` Charvi Mendiratta
2021-02-18 19:36   ` Junio C Hamano [this message]
2021-02-19  6:14     ` Charvi Mendiratta

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqq8s7lf9ra.fsf@gitster.g \
    --to=gitster@pobox.com \
    --cc=charvi077@gmail.com \
    --cc=christian.couder@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=phillip.wood123@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.