Page MenuHomePhabricator

Remove unnecessary "," from Pylint version regex
ClosedPublic

Authored by epriestley on May 8 2019, 5:21 PM.
Tags
None
Referenced Files
F18875847: D20505.id.diff
Nov 5 2025, 7:44 PM
F18867643: D20505.diff
Nov 4 2025, 1:39 AM
F18818042: D20505.diff
Oct 21 2025, 8:31 PM
F18791271: D20505.id.diff
Oct 16 2025, 8:16 AM
F18789222: D20505.id48898.diff
Oct 15 2025, 8:38 AM
F18748269: D20505.id.diff
Oct 3 2025, 8:19 PM
F18735427: D20505.diff
Oct 1 2025, 1:35 AM
F18711010: D20505.diff
Sep 29 2025, 3:32 AM
Subscribers
None

Details

Summary

See PHI1238. This "," in the regex can only make the lint binding fail if there is no "," in the version output string. In modern versions of Pylint, there is (apparently) no comma in the version string. Remove it.

See also https://discourse.phabricator-community.org/t/arcanistpylintlinter-version-regex-issue/2688

Test Plan
$ pip install pylint
Traceback (most recent call last):
  File "/Users/epriestley/Library/Python/2.7/bin/pip", line 6, in <module>
    from pip._internal import main
ImportError: No module named pip._internal

¯\_(ツ)_/¯

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable