Page MenuHomePhabricator

losen pep8 version check
ClosedPublic

Authored by cburroughs on Feb 10 2015, 3:23 AM.
Tags
None
Referenced Files
F15415763: D11728.diff
Thu, Mar 20, 7:48 AM
Unknown Object (File)
Feb 20 2025, 9:16 PM
Unknown Object (File)
Feb 15 2025, 7:26 AM
Unknown Object (File)
Feb 10 2025, 3:36 AM
Unknown Object (File)
Jan 29 2025, 12:03 AM
Unknown Object (File)
Jan 28 2025, 9:31 PM
Unknown Object (File)
Jan 27 2025, 10:54 PM
Unknown Object (File)
Jan 22 2025, 5:26 AM

Details

Summary

pep8 has used both 2 (1.2) and 3 (1.2.1) digit versions. Losen
the version check to allow for both.

NOTE: This is the same regex as flake8.
Test Plan

arc unit with a 2 and 3 digit pep8 version on $PATH.

Diff Detail

Repository
rARC Arcanist
Branch
pep8-version
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 4418
Build 4432: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed

TimeTest
122 msArcanistRubyLinterTestCase::testLinter
60 msArcanistChmodLinterTestCase::testLinter
0 msArcanistClosureLinterTestCase::testVersion
51 msArcanistCommitLinterTestCase::testLinter
0 msArcanistCpplintLinterTestCase::testVersion
View Full Test Results (1 Failed · 22 Passed · 28 Skipped)

Event Timeline

cburroughs retitled this revision from to losen pep8 version check.
cburroughs updated this object.
cburroughs edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Feb 10 2015, 2:17 PM
This revision was automatically updated to reflect the committed changes.