facebook/arcanist — pull request #43 by rafikk (2012-07-19 17:44:27+00:00)
Title: Specify error severity for some PyFlakes messages

Since PyFlakes has no internal demarcation of errors and warnings, running arc lint without the `lintall` option specified results in missing syntax errors and other important messages since ArcanistPyFlakesLinter specifies all messages as `ArcanistLintSeverity::SEVERITY_WARNING`. This pull request marks some messages as SEVERITY_ERROR based on an internal regex. There was some talk of making this a user-specified option in .arcconfig, but this, to me, puts too much onus on individual developers. I'd be open to making it overridable through arcconfig, but I think the linter itself should have some opinion about what constitutes an error.