[webkit-dev] Removal of trailing whitespace
rniwa at apple.com
Wed Apr 12 10:08:21 PDT 2023
WebKi proejctt’s long term policy has been to not do this:
I don’t think we should change that.
- R. Niwa
> On Apr 12, 2023, at 9:17 AM, Chris Dumez via webkit-dev <webkit-dev at lists.webkit.org> wrote:
> I am against this because it adds a lot of noise to patches I am trying to review.
> I have seen PRs where white space changes account for more than half the patch I am trying to review.
> Dropping trailing spaces on the lines you’re modifying is OK but in the whole file is too noisy IMO.
>> On Apr 12, 2023, at 1:22 AM, Anne van Kesteren via webkit-dev <webkit-dev at lists.webkit.org> wrote:
>> To reduce the overhead of switching between projects with different
>> whitespace requirements, I would like to suggest we start being
>> lenient when trailing whitespace is removed. In particular when a file
>> is being changed to fix a bug.
>> I could see going even further and enforcing this via the style
>> checker, if there is appetite for that.
>> Thanks for considering!
>> webkit-dev mailing list
>> webkit-dev at lists.webkit.org
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the webkit-dev