b828910188
Introduced changes: - pre-commit config and rules. - Add pre-commit to pep8 gate, Flake8 is covered in the pre-commit hooks. - Applying fixes for pre-commit compliance in all code. Also commit hash will be used instead of version tags in pre-commit to prevend arbitrary code from running in developer's machines. pre-commit will be used to: - trailing whitespace; - Replaces or checks mixed line ending (mixed-line-ending); - Forbid files which have a UTF-8 byte-order marker (check-byte-order-marker); - Checks that non-binary executables have a proper shebang (check-executables-have-shebangs); - Check for files that contain merge conflict strings (check-merge-conflict); - Check for debugger imports and py37+ breakpoint() calls in python source (debug-statements); - Attempts to load all yaml files to verify syntax (check-yaml); - Run flake8 checks (flake8) (local) For further details about tests please refer to: https://github.com/pre-commit/pre-commit-hooks Change-Id: I8c993d6ffaf869f67713bbbde8632a0143ed0c72 Signed-off-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
36 lines
1.3 KiB
YAML
36 lines
1.3 KiB
YAML
# We from the Oslo project decided to pin repos based on the
|
|
# commit hash instead of the version tag to prevend arbitrary
|
|
# code from running in developer's machines. To update to a
|
|
# newer version, run `pre-commit autoupdate` and then replace
|
|
# the newer versions with their commit hash.
|
|
|
|
default_language_version:
|
|
python: python3
|
|
|
|
repos:
|
|
- repo: https://github.com/pre-commit/pre-commit-hooks
|
|
rev: ebc15addedad713c86ef18ae9632c88e187dd0af # v3.1.0
|
|
hooks:
|
|
- id: trailing-whitespace
|
|
# Replaces or checks mixed line ending
|
|
- id: mixed-line-ending
|
|
args: ['--fix', 'lf']
|
|
exclude: '.*\.(svg)$'
|
|
# Forbid files which have a UTF-8 byte-order marker
|
|
- id: check-byte-order-marker
|
|
# Checks that non-binary executables have a proper shebang
|
|
- id: check-executables-have-shebangs
|
|
# Check for files that contain merge conflict strings.
|
|
- id: check-merge-conflict
|
|
# Check for debugger imports and py37+ breakpoint()
|
|
# calls in python source
|
|
- id: debug-statements
|
|
- id: check-yaml
|
|
files: .*\.(yaml|yml)$
|
|
- repo: https://gitlab.com/pycqa/flake8
|
|
rev: 181bb46098dddf7e2d45319ea654b4b4d58c2840 # 3.8.3
|
|
hooks:
|
|
- id: flake8
|
|
additional_dependencies:
|
|
- hacking>=3.0.1,<3.1.0
|