search

ansible / vscode-ansible

vscode/vscodium extension for providing Ansible auto-completion and integrating quality assurance tools like ansible-lint, ansible syntax check, yamllint, molecule and ansible-test.
https://ansible.readthedocs.io/projects/vscode-ansible/
MIT License
360 stars 86 forks source link

State minimum required version of Ansible Lint #753

Open mforkel opened 1 year ago

mforkel commented 1 year ago

Problem

It seems the extension requires at least version 5.0 of ansible-lint. This should be stated in the documentation. Currently, Ansible 2.9+ and Ansible Lint are listed as requirements.

Solution

Add the minimum required version for Ansible Lint in the requirements section of README.md

Alternatives

No response

Additional context

The version of ansible-lint included in Debian Bullseye is 4.3.7

ssbarnea commented 1 year ago

I agree, we should at least give a clear error when ansible-lint is older than the minimum version we require and disable its integration, likely a WARN even ERROR statusbar badge in that case would be appropriate.

What is even more interesting is that the requirement on ansible-lint will also determine the minimal version of ansible being supported as ansible-lint requires that too. Still, whenever one of them is older than supported we must warn the user.