Ansible-lint: Difference between revisions
From DWIKI
(Created page with "=FAQ= ==Don't compare to literal True/False== Instead of when: (foo == 1 and bar == 0) == false use when: not (foo == 1 and bar == 0) ==Commands should not change things...") |
m (→FAQ) |
||
| (2 intermediate revisions by the same user not shown) | |||
| Line 1: | Line 1: | ||
=FAQ= | =FAQ= | ||
==Don't compare to empty string== | |||
* Use `when: var | length > 0` instead of `when: var != ""`. | |||
* Use `when: var | length == 0` instead of `when: var == ""`. | |||
==Don't compare to literal True/False== | ==Don't compare to literal True/False== | ||
| Line 10: | Line 13: | ||
Sometimes fixed by adding | Sometimes fixed by adding | ||
changed_when: false | changed_when: false | ||
==Lines should be no longer than 160 chars== | |||
ignore for now? | |||
==Shells that use pipes should set the pipefail option== | |||
shell: set -o pipefail && ... | |||
Latest revision as of 17:46, 23 February 2024
FAQ
Don't compare to empty string
- Use `when: var | length > 0` instead of `when: var != ""`.
- Use `when: var | length == 0` instead of `when: var == ""`.
Don't compare to literal True/False
Instead of
when: (foo == 1 and bar == 0) == false
use
when: not (foo == 1 and bar == 0)
Commands should not change things if nothing needs doing
Sometimes fixed by adding
changed_when: false
Lines should be no longer than 160 chars
ignore for now?
Shells that use pipes should set the pipefail option
shell: set -o pipefail && ...
