b6611fd03a
Flake8 went broken recently, adopts correct calling method to avoid regression and fixes broken rules. Also removes openstack-tox-pep8 from gate which was a guaranteed failure when triggered as pep8 was removed. Change-Id: Iba03d3051952f461bb16299a6d32475b2be43041
50 lines
1.7 KiB
YAML
50 lines
1.7 KiB
YAML
---
|
|
repos:
|
|
- repo: https://github.com/pre-commit/pre-commit-hooks
|
|
rev: v2.4.0
|
|
hooks:
|
|
- id: end-of-file-fixer
|
|
- 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: check-yaml
|
|
files: .*\.(yaml|yml)$
|
|
- repo: https://gitlab.com/pycqa/flake8
|
|
rev: '3.7.9'
|
|
hooks:
|
|
- id: flake8
|
|
# commented to allow progressive adoption in smaller patches
|
|
# - repo: https://github.com/adrienverge/yamllint.git
|
|
# rev: v1.18.0
|
|
# hooks:
|
|
# - id: yamllint
|
|
# files: \.(yaml|yml)$
|
|
# types: [file, yaml]
|
|
# entry: yamllint --strict -f parsable
|
|
- repo: https://github.com/ansible/ansible-lint.git
|
|
rev: v4.1.1a6
|
|
hooks:
|
|
- id: ansible-lint
|
|
always_run: true
|
|
pass_filenames: false
|
|
# do not add file filters here as ansible-lint does not give reliable
|
|
# results when called with individual files.
|
|
# https://github.com/ansible/ansible-lint/issues/611
|
|
verbose: true
|
|
entry: env ANSIBLE_LIBRARY=./library ansible-lint --force-color -p -v
|
|
- repo: https://github.com/openstack-dev/bashate.git
|
|
rev: 0.6.0
|
|
hooks:
|
|
- id: bashate
|
|
entry: bashate --error . --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)
|