Re: [PATCH 1/2] rebase -i: reword in-editor documentation of "exec"
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 20, 2011, 20:59 UTC
- Message-ID
- <7vy66fqoji.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <20110120200949.GB11702@burratino>
Jonathan Nieder <jrnieder@gmail.com> writes:
> Maybe "exec" should be explained outside this table? For example, > maybe something along the lines of > > x, exec = run an arbitrary command (see below)
Ok, none of the other insns in the insn sheet mention what the argument to the command means anyway (e.g. "p, pick = replay the commit" doesn't say explicitly where the commit comes from), so I think the original patch is probably fine.
If we wanted to be more helpful, perhaps s/(see below)/specified on the rest of the line/ should be sufficient without adding extra lines.