WebKi proejctt’s long term policy has been to not do this: https://lists.webkit.org/pipermail/webkit-dev/2009-August/009665.html 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@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.
Chris.
On Apr 12, 2023, at 1:22 AM, Anne van Kesteren via webkit-dev <webkit-dev@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@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev