Page MenuHomePhabricator

Fix pyflakes tests for recent pyflakes versions
ClosedPublic

Authored by jrtc27 on Mon, Jan 11, 4:03 AM.

Details

Summary

Since 2.1.0 (commit 75bc0c03c145), pyflakes has included the Python
version and platform in its version output, so ignore it if present.

Since 2.2.0 (commit 6ba3f8e0b59b), pyflakes has included the column
number in its messages, so update the parser to include it and drop the
column number from the (only) test in order to work with both old and
new versions. Whilst here, assign names to the capture groups to make
the code clearer.

Test Plan

Ran arc unit

Diff Detail

Repository
rARC Arcanist
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

See T10038 for general context.

src/lint/linter/ArcanistPyFlakesLinter.php
38–39

Ideally, keep this under 80 characters (e.g., by putting $pattern = ... on a separate line).

This revision is now accepted and ready to land.Mon, Jan 11, 4:09 AM

Stay under 80 characters (and format the other long preg_match a bit more nicely)