c20b778cc1
For the reasons inline. Inspired by I0fdd4a79b886d1c4875e748b08f99da809d9d767 Change-Id: I11f78ac0ede9a36e2c9e4abdc9763c0843028387
48 lines
1.4 KiB
Plaintext
48 lines
1.4 KiB
Plaintext
parseable: true
|
|
# ANSIBLE0010: Package installs should not use latest
|
|
# We often deploy latest pip packages
|
|
#
|
|
# 204: Lines < 160
|
|
# Things like keys, script lines, etc exceed this. We have good
|
|
# taste and only use long lines where appropriate.
|
|
#
|
|
# 206: {{var}} should have spaces {{ var }}
|
|
# This rule seems sane, but it appears there's no way to convince
|
|
# ansible-lint perfectly good things that are *not* ansible (like
|
|
# exim configs) in YAML shouldn't have to follow this rule. See
|
|
# https://github.com/ansible/ansible-lint/issues/534; might be able
|
|
# to turn on if fixed.
|
|
#
|
|
# 306: shells with pipes should use pipefail
|
|
# This requires running the shell as /bin/bash. There is a small
|
|
# possibility to hide errors, but we can rely on
|
|
# developers to decide when to explicitly check for failures.
|
|
#
|
|
# 503: tasks that run when "changed" should likely be handlers
|
|
# We can trust reviewers to decide if something should be a
|
|
# generic handler or not.
|
|
#
|
|
# 106: Role name does not match ``^[a-z][a-z0-9_]+$`` pattern
|
|
# These aren't collections so this doesn't matter
|
|
#
|
|
# 208: file permissions not mentioned, re-evaluate after
|
|
# https://github.com/ansible/ansible-lint/pull/949
|
|
skip_list:
|
|
- ANSIBLE0004
|
|
- ANSIBLE0006
|
|
- ANSIBLE0007
|
|
- ANSIBLE0010
|
|
- ANSIBLE0011
|
|
- ANSIBLE0012
|
|
- ANSIBLE0013
|
|
- ANSIBLE0015
|
|
- '106'
|
|
- '208'
|
|
- '204'
|
|
- '206'
|
|
- '304'
|
|
- '306'
|
|
- '503'
|
|
use_default_rules: true
|
|
verbosity: 1
|