Re: [PATCH v3 2/3] reset: add new reset.quiet config setting

From: Ben Peart <peartben@gmail.com>
Date: Oct 23, 2018, 17:31 UTC
Message-ID: <6feb67a1-17fa-4280-9e31-963b619fa051@gmail.com>
In-Reply-To: <20181022200601.GC9917@sigill.intra.peff.net>

On 10/22/2018 4:06 PM, Jeff King wrote:

Show 19 quoted lines

> On Mon, Oct 22, 2018 at 08:13:32PM +0100, Ramsay Jones wrote:
> 
>>>   -q::
>>>   --quiet::
>>> -	Be quiet, only report errors.
>>> +--no-quiet::
>>> +	Be quiet, only report errors. The default behavior respects the
>>> +	`reset.quiet` config option, or `--no-quiet` if that is not set.
>>
>> Sorry, I can't quite parse this; -q,--quiet and --no-quiet on the
>> command line (should) trump whatever rest.quiet is set to in the
>> configuration. Is that not the case?
> 
> That is the case, and what was meant by "the default behavior" (i.e.,
> the behavior when none of these is used). Maybe there's a more clear way
> of saying that.
> 
> -Peff
>

Is this more clear?

-q::
--quiet::
--no-quiet::
	Be quiet, only report errors. The default behavior is set by the
	`reset.quiet` config option. `--quiet` and `--no-quiet` will
	overwrite the default behavior.