Fixed a couple linting issues with yamllint and ansible-review

[paulcalabro]

• yamllint: "truthy value should be true or false"
  (Docs: https://github.com/adrienverge/yamllint/blob/master/yamllint/rules/truthy.py)

• ansible-review: "WARN: Best practice 'Use YAML format for tasks and handlers rather than key=value' not met"
  (Docs: https://github.com/willthames/ansible-review/blob/2aacd7462f6a8a96165d9218122171b74de54b13/lib/ansiblereview/tasks.py)

• ansible-review: "Shell should only be used when essential"
  (Docs: https://github.com/willthames/ansible-review/blob/2aacd7462f6a8a96165d9218122171b74de54b13/lib/ansiblereview/examples/standards.py#L95)

[paulcalabro]

In regards to the shell substitution, no piping is occurring so it's unnecessary.

If you want to execute a command securely and predictably, it may be better to use the command module instead. Best practices when writing playbooks will follow the trend of using command unless the shell module is explicitly required. When running ad-hoc commands, use your best judgement.

(Source: https://docs.ansible.com/ansible/latest/modules/shell_module.html)

[cadoming]

Hi @paulcalabro,

We have tested your changes and they work correctly, soon we will merge the code. Thank you very much for your help and for improving the repository.

[paulcalabro]

👍