search

StackStorm / ansible-st2

Ansible Roles and Playbooks to deploy StackStorm
https://galaxy.ansible.com/StackStorm/stackstorm/
Apache License 2.0
100 stars 75 forks source link

New linting rules #221

Closed LindsayHill closed 5 years ago

LindsayHill commented 5 years ago

ansible-lint 4.0.0 introduces many new default rules. These trip us up in many places.

The only new default rule I have not fixed is the 120-char line length rule. For now I have just excluded that rule from checks.

I have also fixed a small issue with our CircleCi config, where it was running ansible-lint against non-existent *.yaml files.

LindsayHill commented 5 years ago

Thanks for the input both of you. Have merged your suggestions. Now just have to wait for Travis to do its thing. Assuming all is well, I will then merge