Merge "Use pre-commit to run code style checks"

.pre-commit-config.yaml

@@ -0,0 +1,25 @@
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v5.0.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://opendev.org/openstack/hacking
+    rev: 7.0.0
+    hooks:
+      - id: hacking
+        additional_dependencies: []

doc/source/conf.py

@@ -25,7 +25,7 @@ extensions = [
     'sphinx.ext.autodoc',
     'sphinxcontrib.autoprogram',
     'reno.sphinxext',
-    #'sphinx.ext.intersphinx',
+    # 'sphinx.ext.intersphinx',
 ]
 
 # autodoc generation is a bit aggressive and a nuisance when doing heavy
@@ -82,4 +82,4 @@ latex_documents = [
 ]
 
 # Example configuration for intersphinx: refer to the Python standard library.
-#intersphinx_mapping = {'http://docs.python.org/': None}
+# intersphinx_mapping = {'http://docs.python.org/': None}

test-requirements.txt

@@ -1,4 +1,3 @@
-hacking>=6.1.0,<6.2.0 # Apache-2.0
 coverage>=4.0,!=4.4 # Apache-2.0
 oslotest>=1.10.0 # Apache-2.0
 testtools>=1.4.0 # MIT

tox.ini

@@ -24,7 +24,10 @@ commands =
   stestr slowest
 
 [testenv:pep8]
-commands = flake8 {posargs}
+deps =
+    pre-commit>=2.6.0 # MIT
+commands =
+    pre-commit run -a
 
 [testenv:venv]
 commands = {posargs}