PR#34: Make flake8 mandatory for tox. - freshmaker

freshmaker

#34 Make flake8 mandatory for tox.

Merged 8 years ago by jkaluza. Opened 8 years ago by ralph.

mandatory-flake8-in-tox into master

Make flake8 mandatory for tox.

Ralph Bean • 8 years ago

b4de543

tox.ini

file modified

+1 -2

		`@@ -27,8 +27,7 @@`
		`basepython = python3`
		`skip_install = true`
		`deps = flake8`
		`- commands = flake8 --ignore E501,E731`
		`- ignore_outcome = True`
		`+ commands = flake8 --ignore E501,E731 --exclude freshmaker/migrations/,.tox/`

		`[testenv:bandit]`
		`basepython = python3`

ralph commented 8 years ago

This removes the ignore_errors option, which will cause tox to fail on
failing flake8.

The migrations (autogenerated) are ignored as well as the .tox/
directory, which stops the tool from analyzing all of our depednencies
in all of the virtualenvs.

The test currently passes since #32 has been merged.