Re: [PATCH v5] commit: add a commit.verbose config variable

2016-03-14 Thread Eric Sunshine
On Mon, Mar 14, 2016 at 5:29 PM, Eric Sunshine wrote: > By the way, the just-submitted v6 seems to be lacking these new tests > (though, as I said, they are not by any means mandatory). Ignore me. They are present in v6, yet I somehow overlooked them. -- To unsubscribe from this list: send the li

Re: [PATCH v5] commit: add a commit.verbose config variable

2016-03-14 Thread Eric Sunshine
On Mon, Mar 14, 2016 at 3:09 PM, Pranit Bauva wrote: > On Mon, Mar 14, 2016 at 1:54 PM, Eric Sunshine > wrote: >> Not at all mandatory, but it wouldn't hurt to add a couple additional tests: >> >> * commit.verbose=true and --verbose >> * commit.verbose=false and --no-verbose > > I was thinking o

Re: [PATCH v5] commit: add a commit.verbose config variable

2016-03-14 Thread Pranit Bauva
On Mon, Mar 14, 2016 at 1:54 PM, Eric Sunshine wrote: > On Sat, Mar 12, 2016 at 1:41 AM, Pranit Bauva wrote: >> Add commit.verbose configuration variable as a convenience for those >> who always prefer --verbose. >> >> Signed-off-by: Pranit Bauva >> Mentored-by: Eric Sunshine > More typical wo

Re: [PATCH v5] commit: add a commit.verbose config variable

2016-03-14 Thread Eric Sunshine
On Sat, Mar 12, 2016 at 1:41 AM, Pranit Bauva wrote: > Add commit.verbose configuration variable as a convenience for those > who always prefer --verbose. > > Signed-off-by: Pranit Bauva > Mentored-by: Eric Sunshine More typical would be to say Helped-by: rather than Mentored-by:. Also, place

[PATCH v5] commit: add a commit.verbose config variable

2016-03-11 Thread Pranit Bauva
Add commit.verbose configuration variable as a convenience for those who always prefer --verbose. Signed-off-by: Pranit Bauva Mentored-by: Eric Sunshine --- The previous versions of this patch are: - [v4] $gmane/288652 - [v3] $gmane/288634 - [v2] $gmane/288569 - [v1] $gmane/287540 The chan