8ac74a8177
Follows the same linting configuration that was implemented first in tripleo-quickstart-extras which makes used of pre-commit tool for managing all linters. This also avoids problems where a new linter release may break our gates because pre-commit always pins versions. Removes ansible from requirements.txt as it needs to be listed only in ansible-requirements.txt Change-Id: Ia229d3d58763d743bd19ad9099d7907561f3c77f Depends-On: https://review.openstack.org/#/c/626000/
42 lines
1.4 KiB
YAML
42 lines
1.4 KiB
YAML
---
|
|
repos:
|
|
- repo: https://github.com/pre-commit/pre-commit-hooks
|
|
rev: v2.0.0
|
|
hooks:
|
|
- id: trailing-whitespace
|
|
- id: mixed-line-ending
|
|
- id: check-byte-order-marker
|
|
- id: check-executables-have-shebangs
|
|
- id: check-merge-conflict
|
|
- id: debug-statements
|
|
- id: flake8
|
|
additional_dependencies:
|
|
- hacking<1.2.0,>=1.1.0
|
|
- id: check-yaml
|
|
files: .*\.(yaml|yml)$
|
|
- repo: https://github.com/adrienverge/yamllint.git
|
|
rev: v1.13.0
|
|
hooks:
|
|
- id: yamllint
|
|
files: \.(yaml|yml)$
|
|
types: [file, yaml]
|
|
entry: yamllint --strict -f parsable
|
|
- repo: https://github.com/ansible/ansible-lint
|
|
rev: v4.0.0
|
|
hooks:
|
|
- id: ansible-lint
|
|
files: \.(yaml|yml)$
|
|
entry: ansible-lint --force-color -v
|
|
- repo: https://github.com/openstack-dev/bashate.git
|
|
rev: 0.6.0
|
|
hooks:
|
|
- id: bashate
|
|
entry: bashate --error . --verbose --ignore=E006,E040
|
|
# Run bashate check for all bash scripts
|
|
# Ignores the following rules:
|
|
# E006: Line longer than 79 columns (as many scripts use jinja
|
|
# templating, this is very difficult)
|
|
# E040: Syntax error determined using `bash -n` (as many scripts
|
|
# use jinja templating, this will often fail and the syntax
|
|
# error will be discovered in execution anyway)
|