---
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
  # disable experimental rules
  - experimental
  # Package installs should not use latest
  - package-latest
  # Most files should not contain tabs
  - no-tabs
  # 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
  # TODO(frickler): Discuss these in detail, skipping for now to unblock things
  - key-order[task]
  - no-free-form
  - name[play]
  - var-naming[no-role-prefix]
  - risky-file-permissions
  - risky-shell-pipe
  - command-instead-of-shell
  - command-instead-of-module
  - ignore-errors
  - jinja[spacing]