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. Chris. > On Apr 12, 2023, at 1:22 AM, Anne van Kesteren via webkit-dev > <[email protected]> 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 > [email protected] > https://lists.webkit.org/mailman/listinfo/webkit-dev
_______________________________________________ webkit-dev mailing list [email protected] https://lists.webkit.org/mailman/listinfo/webkit-dev
