[Gluster-devel] Patch merged in master branch for coding policy and patch submission

Harshavardhana harsha at harshavardhana.net
Sat Sep 20 01:19:50 UTC 2014


>>
>> What do we do whan the code surrounding the change has tabs? Should we
>> adapt to the local style, or change everything?
>
> It ignores changes in certain types of files, but .c and .h shouldn't have tabs
>

So there are two possibilities

- Whitespace
- erroneous spaces between functions
- Malformed sign-off
- Wrong jenkins URL

are hard errors and you wouldn't be allowed to submit. NOTE: yes it
makes sure to ignore Markdown files which require white-space as their
return character.

Majority of them are treated as "Warning", like "tabs" so which can be
safely ignored or feel to fix them, a choice is given (y/n)

-- 
Religious confuse piety with mere ritual, the virtuous confuse
regulation with outcomes


More information about the Gluster-devel mailing list