Re: [PATCH 1/3] revert: rename --reset option to --quit
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Nov 21, 2011, 20:36 UTC
- Message-ID
- <7vpqglfc1k.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <20111120094824.GC2278@elie.hsd1.il.comcast.net>
Jonathan Nieder <jrnieder@gmail.com> writes:
Show 14 quoted lines
> The option to "git cherry-pick" and "git revert" to discard the > sequencer state introduced by v1.7.8-rc0~141^2~6 (revert: Introduce > --reset to remove sequencer state, 2011-08-04) has a confusing name. > Change it now, while we still have the time. > > Mechanics: > > This commit removes the "git cherry-pick --reset" option. Hopefully > nobody was using it. If somebody was, we can add it back again as a > synonym. > > The new name for "cherry-pick, please get out of my way, since I've > long forgotten about the sequence of commits I was cherry-picking when > you wrote that old .git/sequencer directory" is --quit.
Wouldn't it match other commands better if we called this --abort instead of --quit?
Other than that I think I agree with the reasoning (and I think I too had encountered the irritation with the "sequencer state").