Re: [PATCH] doc/for-each-ref: explicitly specify option names

2017-09-01 Thread Junio C Hamano
Kevin Daudt writes: > For count, sort and format, only the argument names were listed under > OPTIONS, not the option names. > > Add the option names to make it clear the options exist > > Signed-off-by: Kevin Daudt > --- > Documentation/git-for-each-ref.txt | 18 +- > 1 file ch

Re: [PATCH] doc/for-each-ref: explicitly specify option names

2017-09-01 Thread Jonathan Nieder
Kevin Daudt wrote: > For count, sort and format, only the argument names were listed under > OPTIONS, not the option names. > > Add the option names to make it clear the options exist nit: missing full-stop (.) at end of sentence. > Signed-off-by: Kevin Daudt > --- > Documentation/git-for-each

Re: [PATCH] doc/for-each-ref: explicitly specify option names

2017-09-01 Thread Johannes Schindelin
Hi Kevin, On Fri, 1 Sep 2017, Kevin Daudt wrote: > For count, sort and format, only the argument names were listed under > OPTIONS, not the option names. > > Add the option names to make it clear the options exist > > Signed-off-by: Kevin Daudt > --- This patch makes sense to me. Thanks, Joh

Re: [PATCH] doc/for-each-ref: explicitly specify option names

2017-09-01 Thread Jeff King
On Fri, Sep 01, 2017 at 04:49:31PM +0200, Kevin Daudt wrote: > For count, sort and format, only the argument names were listed under > OPTIONS, not the option names. > > Add the option names to make it clear the options exist Yeah, this looks much better. I could see having a general "" section

[PATCH] doc/for-each-ref: explicitly specify option names

2017-09-01 Thread Kevin Daudt
For count, sort and format, only the argument names were listed under OPTIONS, not the option names. Add the option names to make it clear the options exist Signed-off-by: Kevin Daudt --- Documentation/git-for-each-ref.txt | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-)