Re: [PATCH v6 07/27] checkout: inform the user when removing branch state
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Jul 4, 2019, 23:57 UTC
- Message-ID
- <CACsJy8Agbvn0OYiM8z_r6PAJW0Tov9eQyN3NviD8rLQKNdc45A@mail.gmail.com>
- In-Reply-To
- <xmqqa7dw4966.fsf@gitster-ct.c.googlers.com>
On Wed, Jul 3, 2019 at 12:51 AM Junio C Hamano <gitster@pobox.com> wrote:
Show 16 quoted lines
> > Duy Nguyen <pclouds@gmail.com> writes: > > > I agree, this is not "canceling". I think this series causes conflicts > > with pw/clean-sequencer-state-upon-final-commit and the warning is > > accidentally enabled (partly my fault since I named the argument > > "verbose"). > > > > Junio, in this conflict resolution (merging nd/switch-and-restore to > > next), we should pass '0' instead of 'verbose' to > > sequencer_post_commit_cleanup(). > > Thanks for an update to the merge-fix. What should the name of the > parameter to the sequencer_post_commit_cleanup() funcion be then, > though? Perhaps we'd want an update after the topic graduates to > the 'master' branch.
Definitely. I was thinking something with "warn" and "cancel", which is more the the point. Probably warn_cancel_in_progress, or something.
-- Duy