Page MenuHomePhabricator
Create Task
Maniphest T7293

Add support for ansible-lint
Open, Needs TriagePublic
Actions

Description

We use Ansible to manage our infrastructure at work, and as we're starting to deploy Arcanist for code reviews, I've written a class derived from ArcanistExternalLinter that adds support for calling ansible-lint.

If there is interest for adding support for this, I'll create a review request for my changes. I've signed the CLA.

Revisions and Commits

rARC Arcanist
Needs ReviewD14662 Add support for Ansible-Lint
AbandonedD11798 Add ansible-lint support

Related Objects

Event Timeline

michel-slm created this task.Feb 17 2015, 11:40 AM
michel-slm raised the priority of this task from to Needs Triage.
michel-slm updated the task description. (Show Details)
michel-slm added a project: Lint.
michel-slm added a project: Arcanist.
michel-slm added a subscriber: michel-slm.
joshuaspence added a subscriber: joshuaspence.Feb 17 2015, 7:44 PM
michel-slm added a comment.Feb 18 2015, 4:21 AM

Patch is currently here - we needed this in production so we wrote it and then looked at the contribution guide later.

For when I create a review request, should there already be a test suite, or can that be added during the review?

michel-slm added a revision: D11798: Add ansible-lint support.Feb 18 2015, 5:41 AM
michel-slm added a revision: D14662: Add support for Ansible-Lint.Dec 4 2015, 4:33 AM
epriestley moved this task from Backlog to Bindings on the Lint board.Dec 22 2015, 12:47 PM
epriestley mentioned this in T10038: Plan the mid-term pathway for unit-test/linter bindings.Dec 22 2015, 12:59 PM
epriestley moved this task from Backlog to Unit/Lint Bindings on the Arcanist board.Nov 14 2016, 6:02 PM
jannet added a subscriber: jannet.Feb 2 2017, 12:25 PM

Is there any plan to add support for this in the near future?

Phabricator · Built by Phacility