Re: [PATCH] rebase -i: remove undocumented '--verify' flag

Matthieu Moy wrote:

Martin von Zweigbergk [off-list ref] writes:

quoted

Remove the undocumented and unused '--verify' flag from interactive
rebase.

I don't think this change is good. If a command has a --no-whatever
flag, one expects the --whatever flag to exist too, even if it's a
no-op.

[...]

I think a better change would be to add a comment like

--verify)
	# no-op, exists because --no-verify exists too.

Shouldn't that be

  OK_TO_SKIP_PRE_REBASE=

instead, so that it undoes the effect of an earlier --no-verify?

	;;

-- 
Thomas Rast
trast@{inf,student}.ethz.ch