5f5db610c6
This patch is the merge of 2 current fixes that need to be solved in the same time. Indeed each issue lock our gates independently. Dropping lower constraints testing We facing errors related to the new pip resolver, this topic was discussed on the ML and QA team proposed to to test lower-constraints [1]. I propose to drop this test because the complexity and recurring pain needed to maintain that now exceeds the benefits provided by this mechanismes. [1] http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019390.html Move flake8 as a pre-commit local target. The goal here is to avoid conflicts between flake8 and hacking version each 2 days. Inspired from nova's approach[1]. The flake8 version to install will be determined by hacking and requirements[2] will stay aligned instead of relying on different versions. [1] https://opendev.org/openstack/nova/src/branch/master/.pre-commit-config.yaml#L26-L35 [2] https://opendev.org/openstack/hacking/src/branch/master/requirements.txt#L1 Co-authored-by: Daniel Bengtsson <dbengt@redhat.com> Change-Id: Ia349eba36d8556d4d2775a83b9a344b2ebeed972
40 lines
1.3 KiB
YAML
40 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: local
|
|
hooks:
|
|
- id: flake8
|
|
name: flake8
|
|
additional_dependencies:
|
|
- hacking>=3.0.1,<3.1.0
|
|
language: python
|
|
entry: flake8
|
|
files: '^.*\.py$'
|
|
exclude: '^(doc|releasenotes|tools)/.*$'
|