Page MenuHomePhabricator

Fix pyflakes tests for recent pyflakes versions
ClosedPublic

Authored by jrtc27 on Jan 11 2021, 4:03 AM.
Tags
None
Referenced Files
F14077371: D21504.id51175.diff
Thu, Nov 21, 10:58 PM
F14076422: D21504.diff
Thu, Nov 21, 5:15 PM
Unknown Object (File)
Sun, Nov 17, 11:28 PM
Unknown Object (File)
Thu, Nov 14, 11:43 AM
Unknown Object (File)
Sat, Nov 9, 5:00 PM
Unknown Object (File)
Tue, Nov 5, 5:31 PM
Unknown Object (File)
Sun, Nov 3, 7:26 PM
Unknown Object (File)
Wed, Oct 30, 11:52 PM
Subscribers

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
Branch
php-8
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsrc/lint/linter/ArcanistPyFlakesLinter.php:38TXT3Line Too Long
Unit
Tests Passed
Build Status
Buildable 24988
Build 34478: arc lint + arc unit

Event Timeline

See T10038 for general context.

src/lint/linter/ArcanistPyFlakesLinter.php
38

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)