Add pre-commit

This implementation was copied from case-utils.

A follow-on patch will apply pre-commit effects.

References:
* casework/CASE-Utilities-Python#37

Signed-off-by: Alex Nelson <[email protected]>

Author: ajnelson-nist

.github/workflows/ci.yml

@@ -32,6 +32,10 @@ jobs:
       uses: actions/setup-python@v2
       with:
         python-version: ${{ matrix.python-version }}
+    - name: Pre-commit Checks
+      run: |
+        pip -q install pre-commit
+        pre-commit run --all-files
     - name: Start from clean state
       run: make clean
     - name: Run tests

.gitignore

@@ -4,6 +4,7 @@
 *.swp
 .generated-*
 .pytest_cache
+.venv-pre-commit
 __pycache__
 build
 catalog-v001.xml

.pre-commit-config.yaml

@@ -0,0 +1,14 @@
+repos:
+  - repo: https://github.com/psf/black
+    rev: 22.6.0
+    hooks:
+      - id: black
+  - repo: https://github.com/pycqa/flake8
+    rev: 4.0.1
+    hooks:
+      - id: flake8
+  - repo: https://github.com/pycqa/isort
+    rev: 5.10.1
+    hooks:
+      - id: isort
+        name: isort (python)

Makefile

@@ -18,6 +18,7 @@ PYTHON3 ?= $(shell which python3.9 2>/dev/null || which python3.8 2>/dev/null ||
 GSED ?= $(shell which gsed 2>/dev/null || which sed)
 
 all: \
+  .venv-pre-commit/var/.pre-commit-built.log \
   README.md
 
 .PHONY: \
@@ -59,8 +60,32 @@ README.md: \
 	  --directory lib
 	touch $@
 
+# This virtual environment is meant to be built once and then persist, even through 'make clean'.
+# If a recipe is written to remove this flag file, it should first run `pre-commit uninstall`.
+.venv-pre-commit/var/.pre-commit-built.log:
+	rm -rf .venv-pre-commit
+	test -r .pre-commit-config.yaml \
+	  || (echo "ERROR:Makefile:pre-commit is expected to install for this repository, but .pre-commit-config.yaml does not seem to exist." >&2 ; exit 1)
+	$(PYTHON3) -m venv \
+	  .venv-pre-commit
+	source .venv-pre-commit/bin/activate \
+	  && pip install \
+	    --upgrade \
+	    pip \
+	    setuptools \
+	    wheel
+	source .venv-pre-commit/bin/activate \
+	  && pip install \
+	    pre-commit
+	source .venv-pre-commit/bin/activate \
+	  && pre-commit install
+	mkdir -p \
+	  .venv-pre-commit/var
+	touch $@
+
 # After running unit tests, see if README.md needs to be regenerated.
 check: \
+  .venv-pre-commit/var/.pre-commit-built.log \
   .lib.done.log
 	$(MAKE) \
 	  PYTHON3=$(PYTHON3) \
@@ -80,7 +105,8 @@ clean:
 	  clean
 
 download: \
-  .lib.done.log
+  .lib.done.log \
+  .venv-pre-commit/var/.pre-commit-built.log
 	$(MAKE) \
 	  PYTHON3=$(PYTHON3) \
 	  --directory tests \

setup.cfg

@@ -25,3 +25,14 @@ python_requires = >=3.7
 [options.entry_points]
 console_scripts =
     case_shacl_inheritance_reviewer = case_shacl_inheritance_reviewer:main
+
+[flake8]
+# https://black.readthedocs.io/en/stable/guides/using_black_with_other_tools.html#flake8
+extend-ignore =
+  E203
+  E302
+  E501
+
+[isort]
+# https://pycqa.github.io/isort/docs/configuration/black_compatibility.html
+profile = black