- User Since
- Apr 17 2012, 11:16 PM (405 w, 2 d)
Dec 17 2013
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...)