Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 3/3] git-apply --whitespace=nowarn

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:42:20

A Large Angry SCM [off-list ref] writes:
Junio C Hamano wrote:
quoted
Andrew insists --whitespace=warn should be the default, and I
tend to agree.  This introduces --whitespace=warn, so if your
project policy is more lenient, you can squelch them by having
apply.whitespace=nowarn in your configuration file.
Signed-off-by: Junio C Hamano <redacted>
I think this is wrong. The default policy of, non-security, tools
SHOULD be the least restrictive and most flexible.
The reason is that --whitespace=warn does not refuse to apply
but spits out some warning messages.  Admittedly, going from
zero, any amount of new warning message makes it infinitely more
chatty, but then people can squelch it by having the
configuration option "apply.whitespace = nowarn".
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help