Re: [PATCH] Add gitattributes file making whitespace checking pickier

Previous thread: Re: [PATCH 1/2] config api: Add git_config_magic_int() by Junio C Hamano on Tuesday, February 12, 2008 - 2:41 pm. (2 messages)

Next thread: Merge-Recursive Improvements by Voltage Spike on Tuesday, February 12, 2008 - 3:16 pm. (14 messages)
From: Junio C Hamano
Date: Tuesday, February 12, 2008 - 2:42 pm

You are quoting an older draft.  The rules applied to 'master'
is actually a bit more lenient:

 - Unless otherwise specified, indent with SP that could be
   replaced with HT are not "bad".  But SP before HT in the
   indent is "bad", and trailing whitespaces are "bad".

 - For C source files, initial indent by SP that can be replaced
   with HT is also "bad".

 - Test scripts in t/ and test vectors in its subdirectories can
   contain anything, so we make it unrestricted for now.

So your particular "indent without HT" is generally accepted,
and considered a violation only in C sources.

I see what you mean by "excempt from all rules" and it is
certainly a valid approach, but I at the same time think minimal
whitespace policy should be there, and flagging trailing spaces
and SP before HT is a good idea.
-

Previous thread: Re: [PATCH 1/2] config api: Add git_config_magic_int() by Junio C Hamano on Tuesday, February 12, 2008 - 2:41 pm. (2 messages)

Next thread: Merge-Recursive Improvements by Voltage Spike on Tuesday, February 12, 2008 - 3:16 pm. (14 messages)