Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reduce ansible-lint skip list #1296

Open
evgeni opened this issue Feb 12, 2021 · 0 comments
Open

reduce ansible-lint skip list #1296

evgeni opened this issue Feb 12, 2021 · 0 comments

Comments

@evgeni
Copy link
Member

evgeni commented Feb 12, 2021

we currently have the following checks skipped, should revisit that as we go:

  • command-instead-of-module # Using command rather than module
  • command-instead-of-shell # Use shell only when shell functionality is required
  • experimental # all rules tagged as experimental
  • git-latest # Git checkouts must contain explicit version
  • meta-no-info # meta/main.yml should contain relevant info
  • no-changed-when # Commands should not change things if nothing needs doing
  • no-handler # Tasks that run when changed should likely be handlers
  • package-latest # Package installs should not use latest
  • experimental

#1298 handles:

  • deprecated-command-syntax # Using command rather than an argument to e.g. file
  • deprecated-local-action # Do not use 'local_action', use 'delegate_to: localhost'
  • literal-compare # Don't compare to literal True/False
  • empty-string-compare # Don't compare to empty string
  • no-jinja-when # No Jinja2 in when
  • no-relative-paths # Doesn't need a relative path in role
  • risky-shell-pipe # Shells that use pipes should set the pipefail option
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant