Page MenuHomePhabricator

Fix pyflakes tests for recent pyflakes versions

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



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

rARC Arcanist
Lint Passed
Tests Passed
Build Status
Buildable 24997
Build 34487: arc lint + arc unit

Event Timeline

See T10038 for general context.


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

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

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