ansible-lint follow ups

[MSevey]

Describe the bug

The following rules need to be enabled which will require some work and testing to make sure that no bugs are introduced.

• no-changed-when
• deprecated-local-action
• risky-shell-pipe
• risky-file-permissions
• command-instead-of-module
• command-instead-of-shell

Screenshots

# This makes linter to fully ignore rules/tags listed below
skip_list:
  - no-changed-when # TODO: enable https://ansible-lint.readthedocs.io/en/latest/default_rules/#no-changed-when
  - deprecated-local-action # TODO: enable https://ansible-lint.readthedocs.io/en/latest/default_rules/#deprecated-local-action
  - risky-shell-pipe # TODO: enable, need to add pipe fail option https://ansible-lint.readthedocs.io/en/latest/default_rules/#risky-shell-pipe
  - risky-file-permissions # TODO: enable, need to work through errors and set all unset file permissions
  - command-instead-of-module # TODO: enable, looks like there are some places to use this module https://docs.ansible.com/ansible/latest/collections/ansible/builtin/git_module.html
  - command-instead-of-shell # TODO: enable, https://ansible-lint.readthedocs.io/en/latest/default_rules/#command-instead-of-shell

Additional context