baf8392455
Allows us to slowly adopt yamllint by having to fix only the files we would be touching. Once we lower the amount of differences we can switch it to run on all files. In addition to having a more consistent code-style this solves *recurring* bugs like duplicate dictionary keys, like the one fixes in this change. Only few weeks ago we had to fix a similar bug. Change-Id: I2ef275700b420a0f700613568e43576079cd4d03
35 lines
1.2 KiB
YAML
35 lines
1.2 KiB
YAML
---
|
|
repos:
|
|
- repo: https://github.com/pre-commit/pre-commit-hooks
|
|
rev: v1.4.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: check-yaml
|
|
files: .*\.(yaml|yml)$
|
|
- repo: https://github.com/adrienverge/yamllint.git
|
|
rev: v1.12.1
|
|
hooks:
|
|
- id: yamllint
|
|
files: \.(yaml|yml)$
|
|
types: [file, yaml]
|
|
# uncommend once a new yamllint release is made:
|
|
# entry: yamllint --strict
|
|
|
|
- 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)
|