e7cc5efd20
The flake8 hook for pre-commit installs its own flake8 which is not what hacking is pinned to. This results in a bunch of errors. By doing a local hook we can cause these to be the same. There are flake8 exclusions in tox.ini that need to be used in pre-commit. Also we should exclude svg files from the whitespace modifications, because who cares. Once those are fixed, running pre-commit -a fixes some whitespace issues. While the normal use isn't to run -a - if someone does make a patch that touches any of these they'll get erroneous unrelated errors compared to their lower work - so fix all of the existing issues, which are mostly space-at-end-of-line issues it seems. Change-Id: I78e753492e99c03b30a0a691db3bd75ee3d289c9
33 lines
895 B
YAML
33 lines
895 B
YAML
---
|
|
repos:
|
|
- repo: https://github.com/pre-commit/pre-commit-hooks
|
|
rev: v2.4.0
|
|
hooks:
|
|
- id: trailing-whitespace
|
|
- id: mixed-line-ending
|
|
args: ['--fix', 'lf']
|
|
exclude: '.*\.(svg)$'
|
|
- id: check-byte-order-marker
|
|
- id: check-executables-have-shebangs
|
|
- id: check-merge-conflict
|
|
- id: debug-statements
|
|
# nova/cmd/manage.py imports pdb on purpose.
|
|
exclude: 'nova/cmd/manage.py'
|
|
- id: check-yaml
|
|
files: .*\.(yaml|yml)$
|
|
- repo: https://github.com/Lucas-C/pre-commit-hooks
|
|
rev: v1.1.7
|
|
hooks:
|
|
- id: remove-tabs
|
|
exclude: '.*\.(svg)$'
|
|
- repo: local
|
|
hooks:
|
|
- id: flake8
|
|
name: flake8
|
|
additional_dependencies:
|
|
- hacking>=1.1.0,<1.2.0
|
|
language: python
|
|
entry: flake8
|
|
files: '^.*\.py$'
|
|
exclude: '^(doc|releasenotes|tools)/.*$'
|