<div dir="ltr">As of bug 1460856, mozlint (the library underpinning our various<div>lint systems) will no longer display warnings by default. All</div><div>current linters which were previously emitting warnings have</div><div>been changed to the 'error' level, so this won't cause any loss</div><div>of coverage. There are different implications depending where the</div><div>lint runs.</div><div><br></div><div><b>CI and vcs hooks</b></div><div>Taskcluster lint tasks and the vcs hooks will not fail if there were</div><div>only warnings. In other words, you won't be backed out for</div><div>warnings.<br></div><div><br></div><div><b>Command line</b><br></div><div>The default |mach lint| formatter will tell you if there are any<br></div><div>suppressed warnings, but will only display them if you pass in</div><div>`-W/--warnings`. E.g:</div><div><br></div><div><b>$</b> ./mach lint -nl codespell python/mozlint <br>✖ 0 problems (0 errors, 1 warning)</div><div><br></div><div><b>$</b> ./mach lint -nl codespell python/mozlint --warnings<br>python/mozlint/mozlint/roller.py<br>  62  warning  retuns  ==> returns  (codespell)<br><br>✖ 1 problem (0 errors, 1 warning)<br></div><div><br></div><div><b>Review</b><br></div><div>There is a follow-up on file</div><div>(<a href="https://github.com/mozilla/release-services/issues/1447" target="_blank">https://github.com/mozilla/release-services/issues/1447</a>)</div><div>to get the Phabricator reviewbot to show warnings by default. So</div><div>warning level issues on your review can be treated as strong</div><div>suggestions, but it is fine to ignore them if you want.</div><div><br></div><div>This change will allow us to enable a subclass of lint issues that</div><div>might contain false positives (e.g the codespell linter), or else</div><div>aren't important enough to cause a backout (but would ideally</div><div>be fixed). This change will allow the reviewbot to be more</div><div>aggressive with the issues it raises, as well as give us a bit of</div><div>a middle ground when enabling a new linter across the tree.<br></div><div><br></div><div>Please let me know if you have any questions or concerns</div><div>about this change!</div><div><br></div><div>Thanks,</div><div>Andrew<br></div></div>