Page MenuHomePhabricator
Feed Advanced Search

Dec 17 2013

koreno added a comment to T3960: Add capability to hide lint messages when viewing a diff.

Excluding files seems to me too harsh.
The lint warnings might be useful for the next iteration, but are just very much in the way of reading the code as it is in this revision to be able to comment on other aspects.
I just find myself having to reject the revision and waste a review cycle on linting...
(Also, in some situations fixing the lint issue would make the code inconsistent with the legacy code around it, which is less desirable...)

Dec 17 2013, 10:17 AM · Restricted Project, Differential