[PATCH] Add utility to check for indentation in files or patches

Simo simo at samba.org
Mon Apr 9 11:52:42 UTC 2018


On Mon, 2018-04-09 at 15:54 +1200, Douglas Bagnall wrote:
> On 09/04/18 11:44, Andrew Bartlett via samba-technical wrote:
> 
> > Is there a way to whitelist/blacklist file types and directories with
> > this tool?
> > 
> > We really don't want commits bouncing because they import a new heimdal
> > or update third_party for example, and it would be helpful to be able
> > to exclude some files both temporarily when (for example) moving old
> > code around.
> 
> If we're talking about a pre-commit hook here, we could make the
> rejection message say something like:
> 
>    Try again with
> 
>      git commit --no-verify
> 
>    if you are sure you are doing the right thing.

The hook can definitely say that, yes.

These utilities work independently from the hook, so it would be weird
to bake it into them.

Simo.



More information about the samba-technical mailing list