feat(*): add make test-style targets

This gives us an opporunity to run code checkers and linters as part
of testing, as we used to on travis-ci.org. By default, unit tests
will check code syntax and style first.

Author: mboersma

Makefile

@@ -41,14 +41,16 @@ coverage:
 	coverage run manage.py test --noinput api
 	coverage html
 
-flake8:
-	flake8
-
 test: test-unit test-functional
 
-test-unit:
+setup-venv:
 	@if [ ! -d venv ]; then virtualenv venv; fi
 	venv/bin/pip install -q -r requirements.txt -r dev_requirements.txt
+
+test-style: setup-venv
+	venv/bin/flake8
+
+test-unit: setup-venv test-style
 	venv/bin/python manage.py test --noinput api
 
 test-functional:

dev_requirements.txt

@@ -19,9 +19,7 @@ coverage>=3.7.1
 docutils>=0.12
 
 # Run "make flake8" to check python syntax and style
-flake8==2.1.0
-pep8==1.4.6
-pyflakes==0.7.3
+flake8==2.2.2
 
 # Used for mocking endpoints
 mock==1.0.1

setup.cfg

@@ -1,4 +1,4 @@
 [flake8]
 max-line-length = 99
-exclude = */api/migrations/*,*/api/south_migrations/*,*/build/*,*/venv/*,*/virtualenv/*,*/scheduler/*,*/templates/*
+exclude = api/migrations,api/south_migrations,templates,venv
 max-complexity = 12