Page MenuHomePhabricator

losen pep8 version check
ClosedPublic

Authored by cburroughs on Feb 10 2015, 3:23 AM.
Tags
None
Referenced Files
F14075867: D11728.id28243.diff
Thu, Nov 21, 1:34 PM
Unknown Object (File)
Wed, Nov 20, 1:32 PM
Unknown Object (File)
Sat, Nov 16, 4:49 AM
Unknown Object (File)
Thu, Nov 14, 9:59 PM
Unknown Object (File)
Tue, Nov 12, 3:07 AM
Unknown Object (File)
Fri, Nov 8, 9:10 PM
Unknown Object (File)
Thu, Nov 7, 11:44 PM
Unknown Object (File)
Fri, Nov 1, 7:17 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.