Page MenuHomePhabricator
Differential D15818

Prefer pip to easy_install
ClosedPublic
Actions

Authored by eadler on Apr 29 2016, 4:57 AM.
Tags
None
Referenced Files
F14085436: D15818.diff
Sat, Nov 23, 11:59 AM
F14081533: D15818.diff
Fri, Nov 22, 8:25 PM
Unknown Object (File)
Sun, Nov 17, 6:23 PM
Unknown Object (File)
Fri, Nov 15, 12:06 AM
Unknown Object (File)
Sun, Nov 10, 12:31 PM
Unknown Object (File)
Tue, Nov 5, 11:48 PM
Unknown Object (File)
Sun, Nov 3, 9:42 PM
Unknown Object (File)
Oct 17 2024, 7:47 PM
View All Files
Subscribers
avivey
Korvin

Details

Reviewers
epriestley
avivey
tjstum
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T10038: Plan the mid-term pathway for unit-test/linter bindings
T10892: Recommend pip instead of easy_install for Python linters
Commits
rARCc58f1b9a2507: Prefer pip to easy_install
Summary

Prefer pip to easy_install
also fixes incorrect install instructions for closure linters

Fixes T10892
Ref T10038

Test Plan

inspection

Diff Detail

Repository
rARC Arcanist
Branch
eax/easy_install (branched from master)
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 11974
Build 15062: Run Core Tests
Build 15061: arc lint + arc unit

Event Timeline

eadler updated this revision to Diff 38100.Apr 29 2016, 4:57 AM
eadler retitled this revision from to Prefer pip to easy_install.
eadler updated this object.
eadler edited the test plan for this revision. (Show Details)
eadler added reviewers: epriestley, tjstum.
eadler added tasks: T10892: Recommend pip instead of easy_install for Python linters, T10038: Plan the mid-term pathway for unit-test/linter bindings.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 29 2016, 4:57 AM
Herald added a subscriber: Korvin. · View Herald Transcript
eadler updated this revision to Diff 38101.Apr 29 2016, 4:59 AM
eadler edited edge metadata.

reupload with fix

avivey accepted this revision.Apr 29 2016, 5:06 AM
avivey added a reviewer: avivey.
avivey added a subscriber: avivey.

lgtm

epriestley accepted this revision.Apr 29 2016, 12:18 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 29 2016, 12:18 PM
tjstum accepted this revision.Apr 29 2016, 2:15 PM
tjstum edited edge metadata.

Awesome, thanks!

Closed by commit rARCc58f1b9a2507: Prefer pip to easy_install (authored by eadler, committed by eadler). · Explain WhyApr 29 2016, 3:45 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 38101

View Options

src/lint/linter/ArcanistClosureLinter.php

Loading...
View Options

src/lint/linter/ArcanistFlake8Linter.php

Loading...
View Options

src/lint/linter/ArcanistPEP8Linter.php

Loading...
Phabricator · Built by Phacility