---
# NOTE(priteau): Rule file imported from kolla-ansible
strict: true
use_default_rules: true
skip_list:
  # [E301] Commands should not change things if nothing needs doing
  # TODO(mnasiadka): Fix tasks that fail this check in a later iteration
  - no-changed-when
  # [E503] Tasks that run when changed should likely be handlers
  - no-handler
  # [unnamed-task] All tasks should be named
  # FIXME(mgoddard): Add names to all tasks
  - unnamed-task
  # Package installs should not use latest
  - package-latest
  # NOTE(frickler): Agreed at Zed PTG not to use FQCN for builtin actions for now, due to
  # conflicts with open patches and backports.
  - fqcn-builtins
  # Allow Jinja templating inside task and play names
  - name[template]
  # FQCNs again, now for module actions
  - fqcn[action]
  # role name check matching ^*$
  - role-name
  # Allow long lines
  - yaml[line-length]
  # TODO(frickler): Discuss these in detail, skipping for now to unblock things
  - command-instead-of-module
  - command-instead-of-shell
  - deprecated-local-action
  - key-order[task]
  - name[play]
  - risky-file-permissions
  - risky-shell-pipe
  - run-once[task]
  - var-naming[no-reserved]
  - var-naming[no-role-prefix]
