d8f31e0a5e
* Reworked tox pep8 into linters job, that runs: - pep8 - bandit - bashate - doc8 - yamllint - ansible-lint (validate-all-files.py + ansible-lint) * Skip E701 - missing galaxy_info in meta and E602 see [1]. * Skip E301 and E503 - followup later in a separate change * Added ansible-role-jobs to zuul.d/project.yaml which will run openstack-tox-linters job in check queue * Fixed remaining style issue * Made tox and docs reference the new env for linters * Dropped pype environment (not supported) [1]: https://github.com/ansible/ansible-lint/issues/457 Change-Id: I494b4b151804aac8173120e6c6e42bc2fdb00234
12 lines
458 B
Plaintext
12 lines
458 B
Plaintext
use_default_rules: true
|
|
skip_list:
|
|
# [E701] galaxy_info missing in metadata
|
|
- '701'
|
|
# [E602] https://github.com/ansible/ansible-lint/issues/457
|
|
- '602'
|
|
# [E301] Commands should not change things if nothing needs doing
|
|
# TODO(mnasiadka): Fix tasks that fail this check in a later iteration
|
|
- '301'
|
|
# [E503] Tasks that run when changed should likely be handlers
|
|
# TODO(mnasiadka): Rework baremetal role to do handlers instead of when: *.changed
|
|
- '503' |