[erlang-patches] What's the whitespace policy for patches?

Henrik Nord henrik@REDACTED
Fri Nov 21 09:48:21 CET 2014


Hello!

Yes it is the first answer below:

1. Don't introduce new bogus whitespace (e.g. trailing spaces),
        but don't fix existing bogus whitespace. We don't want
        unrelated changes in commits, even if it's just whitespace.

You are free to fix whitespace on the lines you are editing, but do not 
make wide sweeps fixing whitespace in areas you did not edit.



On 2014-11-20 22:29, Matthias Lang wrote:
> Hi,
>
> Sorry to bother you with a trivial problem.
>
> What's the policy for whitespace changes in patches? Is it:
>
>      1. Don't introduce new bogus whitespace (e.g. trailing spaces),
>         but don't fix existing bogus whitespace. We don't want
>         unrelated changes in commits, even if it's just whitespace.
>
> or  2. Fixing bogus whitespace is fine, we all use 'git diff -w'.
>         Eventually we'll be rid of it all.
>
> The 'Submitting Patches' wiki answers a related but different question:
>
>    | Check for unnecessary whitespace before committing with git diff --check.
>    https://github.com/erlang/otp/wiki/Submitting-patches
>
> I suspect it should actually say
>
>    | Check for unnecessary whitespace _changes_ before...
>
> otherwise 'git diff --check' wouldn't make sense.
>
> Matt
> _______________________________________________
> erlang-patches mailing list
> erlang-patches@REDACTED
> http://erlang.org/mailman/listinfo/erlang-patches

-- 
/Henrik Nord Erlang/OTP




More information about the erlang-patches mailing list