Page MenuHomePhabricator
Create Task
Maniphest T4757

Update pylint support for newest pylint version (1.1)
Closed, ResolvedPublic
Actions

Description

PyLint integration in Arcanist hardcodes flags to pylint: https://secure.phabricator.com/diffusion/ARC/browse/master/src/lint/linter/ArcanistPyLintLinter.php;f099168aa82c1c1f5a6fd8f8b8f32e9237e7b309$165.

The problem is that in newer versions of pylint, -i flag does not exist anymore. People running easy_install pylint as per arcanist recommendation will get the newest version which is not supported.

Please make arcanist support newer pylint versions. Note that newer pylint versions might have the output formating changed also.

Related Objects

Event Timeline

• marin.radu created this task.Apr 7 2014, 10:18 PM
• marin.radu assigned this task to epriestley.
• marin.radu raised the priority of this task from to Needs Triage.
• marin.radu updated the task description. (Show Details)
• marin.radu added a project: Arcanist.
• marin.radu added a subscriber: • marin.radu.
epriestley added subscribers: ekwan, epriestley.Apr 8 2014, 9:59 PM

◀ Merged tasks: T4263.

epriestley renamed this task from [Arcanist][PyLint]: Update support for newest pylint version (1.1) to Update pylint support for newest pylint version (1.1).Apr 8 2014, 11:38 PM
jonescodes added a subscriber: jonescodes.Apr 17 2014, 7:24 PM
epriestley added a comment.Apr 17 2014, 7:58 PM

These patches both break older versions, so the actual patch needs to be wider reaching than either of those.

epriestley closed this task as Resolved.May 6 2014, 2:10 AM

rARC85000be9 is a probable fix for this.

Phabricator · Built by Phacility