openstack/automaton
Code Issues Proposed changes

Move flake8 as a pre-commit local target.

Browse Source 
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

Change-Id: I402e708832395d95b04a5b956a19d5cc8c3f1c1f
This commit is contained in:
Daniel Bengtsson 2021-03-23 13:00:14 +01:00 committed by Stephen Finucane
parent 406007e43e
commit 80bd6a5442
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
.pre-commit-config.yaml
View File

@ -27,9 +27,13 @@ repos:
- id: debug-statements - id: debug-statements
- id: check-yaml - id: check-yaml
files: .*\.(yaml|yml)$ files: .*\.(yaml|yml)$
- repo: https://gitlab.com/pycqa/flake8 - repo: local
rev: fb91b994ed4adf4f2b4890e7bdba82f57e3a81df # 3.8.4
hooks: hooks:
- id: flake8 - id: flake8
name: flake8
additional_dependencies: additional_dependencies:
- hacking>=3.1.0,<4.0 - hacking>=3.1.0,<4.0
language: python
entry: flake8
files: '^.*\.py$'
exclude: '^(doc|releasenotes|tools)/.*$'