Le 14/12/2018 à 22:55, Kartikaya Gupta a écrit :
On Fri, Dec 14, 2018 at 1:58 PM Sylvestre Ledru<sle...@mozilla.com> wrote:
We have more and more tools at review phase (clang-format, flake8, eslint,
clang-tidy, codespell, etc) which propose some auto-fixes.
Honestly I find it quite annoying when I'm trying to review a patch
and the phabricator diff is filled up with bot comments about
formatting. As a reviewer, I shouldn't have to think about formatting,
and it shouldn't fill up my screen and prevent me from looking at the
actual code.
We received this feedback from several folks and we agree (initially, we were
showing the diff itself but it wasn't
bringing much).
This is fixed in staging and the fix should go in the next release of the
platform (Dec 22nd)
Cheers
S
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
