infra: update the pre-commit hook with linters (#678)

* infra: update the pre-commit hook with linters and secrets check

Co-authored-by: Abe Coull <[email protected]>
Co-authored-by: Cody Wang <[email protected]>

.git-template/hooks/pre-commit

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+if [ -f .pre-commit-config.yaml ]; then
+    echo 'pre-commit configuration detected, but `pre-commit install` was never run' 1>&2
+    exit 1
+fi

.pre-commit-config.yaml

@@ -1,18 +1,8 @@
 repos:
-- repo: local
-  hooks:
-  - id: tox
-    name: tox
-    stages: [push]
-    language: system
-    entry: tox
-    types: [python]
-    pass_filenames: false
-
-  - id: tox_integ_tests
-    name: tox integ tests
-    stages: [push]
-    language: system
-    entry: tox -e integ-tests
-    types: [python]
-    pass_filenames: false
+  - repo: local
+    hooks:
+      - id: tox_linters
+        name: linters
+        entry: tox -e linters_check
+        language: system
+        types: [python]

tox.ini

@@ -41,6 +41,19 @@ commands =
     {[testenv:black]commands}
     {[testenv:flake8]commands}
 
+# Read only linter env
+[testenv:linters_check]
+basepython = python3
+skip_install = true
+deps =
+    {[testenv:isort_check]deps}
+    {[testenv:black_check]deps}
+    {[testenv:flake8]deps}
+commands =
+    {[testenv:isort_check]commands}
+    {[testenv:black_check]commands}
+    {[testenv:flake8]commands}
+
 [testenv:flake8]
 basepython = python3
 skip_install = true
@@ -60,6 +73,14 @@ deps =
 commands =
     isort . {posargs}
 
+[testenv:isort_check]
+basepython = python3
+skip_install = true
+deps =
+    isort
+commands =
+    isort . -c {posargs}
+
 [testenv:black]
 basepython = python3
 skip_install = true
@@ -68,6 +89,14 @@ deps =
 commands =
     black ./ {posargs}
 
+[testenv:black_check]
+basepython = python3
+skip_install = true
+deps =
+    black
+commands =
+    black --check . {posargs}
+
 [testenv:docs]
 basepython = python3
 deps =