This commit adds support for Ansible-Lint, using its PEP8-mode output for
easy parsing.
Unit tests are included for all the currently-supported lint checks.
Details
Details
- Reviewers
- None
- Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T7293: Add support for ansible-lint
pip install --user ansible-lint cd src/lint/linters/__tests__/ arc unit ArcanistAnsibleLintLinterTestCase.php
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- ansible-lint
- Lint
Lint Passed Severity Location Code Message Auto-Fix src/lint/linter/__tests__/ansible-lint/trailing_whitespace.lint-test:1 TXT6 Trailing Whitespace - Unit
Tests Passed - Build Status
Buildable 9277 Build 11004: arc lint + arc unit
Event Timeline
Comment Actions
I tried setting this up locally but arc lint isn't throwing warning when I hit a SEVERITY_WARNING from the ansible-lint. Is that a known issue to you @michel-slm or could it be that I'm doing something wrong?
| src/lint/linter/ArcanistAnsibleLintLinter.php | ||
|---|---|---|
| 82 | I had to add brackets around the ANSIBLE0003 to get a proper match: if (preg_match('/^\[ANSIBLE0003\]$/', $code)) { | |