Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix ansible-lint linter definition. #3601

Merged
merged 2 commits into from
Feb 28, 2021
Merged

Conversation

serapas
Copy link
Contributor

@serapas serapas commented Feb 20, 2021

Use ansible-lint's feature auto-detection instead of temporary file.
For auto-detection to work, ansible project has to also be a git repository.

Don't use yaml rules. These are checked by yamllint.

Refactor pattern to work with ansible-lint >=5.0 version.

Clean-up obsolete test cases.

Use ansible-lint's feature auto-detection instead of temporary file.
For auto-detection to work, ansible project has to be also a git repository.

Don't use yaml rules. These are checked by yamllint.

Refactor pattern to work with ansible-lint >=5.0 version.

Clean-up obsolete test cases.
ale_linters/ansible/ansible_lint.vim Show resolved Hide resolved
ale_linters/ansible/ansible_lint.vim Show resolved Hide resolved
ale_linters/ansible/ansible_lint.vim Outdated Show resolved Hide resolved
test/handler/test_ansible_lint_handler.vader Show resolved Hide resolved
ale_linters/ansible/ansible_lint.vim Outdated Show resolved Hide resolved
Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution.

@hsanson hsanson enabled auto-merge (squash) February 28, 2021 12:42
@hsanson hsanson merged commit 038e4a8 into dense-analysis:master Feb 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants