Thomas Rast <trast@xxxxxxxxxxxxxxx> writes: > Matthieu Moy wrote: >> Martin von Zweigbergk <martin.von.zweigbergk@xxxxxxxxx> writes: >> >> > 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? Yes, right. Useful when an alias contains --no-whatever in particular. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html