Run black --check as part of CI to enforce code style

jstasiak · jstasiak · commit 12477c954e7f · 2019-05-15T23:26:38.000+02:00
diff --git a/.travis.yml b/.travis.yml
@@ -11,6 +11,7 @@ install:
     - pip install -r requirements-dev.txt
     # mypy can't be installed on pypy
     - if [[ "${TRAVIS_PYTHON_VERSION}" != "pypy"* ]] ; then pip install mypy ; fi
+    - if [[ "${TRAVIS_PYTHON_VERSION}" != *"3.5"* ]] ; then pip install black ; fi
 script:
     - make ci
 after_success:
diff --git a/Makefile b/Makefile
@@ -1,11 +1,16 @@
 .PHONY: all virtualenv
 MAX_LINE_LENGTH=110
 PYTHON_IMPLEMENTATION:=$(shell python -c "import sys;import platform;sys.stdout.write(platform.python_implementation())")
+PYTHON_VERSION:=$(shell python -c "import sys;sys.stdout.write('%d.%d' % sys.version_info[:2])")
 
 LINT_TARGETS:=flake8
+
 ifneq ($(findstring PyPy,$(PYTHON_IMPLEMENTATION)),PyPy)
 	LINT_TARGETS:=$(LINT_TARGETS) mypy
 endif
+ifneq ($(findstring 3.5,$(PYTHON_VERSION)),3.5)
+	LINT_TARGETS:=$(LINT_TARGETS) black_check
+endif
 
 
 virtualenv: ./env/requirements.built
@@ -26,6 +31,10 @@ lint: $(LINT_TARGETS)
 flake8:
 	flake8 --max-line-length=$(MAX_LINE_LENGTH) examples *.py
 
+.PHONY: black_check
+black_check:
+	black --check .
+
 mypy:
 	mypy examples/*.py test_zeroconf.py zeroconf.py
 
