User Details
User Details
- User Since
- Apr 17 2012, 11:16 PM (661 w, 3 d)
- Availability
- Available
Dec 17 2013
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...)