Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: tox.ini

Issue 29573900: Noissue - Make tox.ini in python-abp consistent with our other projects (Closed)
Patch Set: Created Oct. 11, 2017, 8:14 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tox.ini
===================================================================
--- a/tox.ini
+++ b/tox.ini
@@ -1,24 +1,34 @@
[tox]
-envlist = py27, py35, py36, pypy, coverage
+envlist = py27,py35,py36,pypy,coverage,flake8
Sebastian Noack 2017/10/11 20:28:51 In no other tox.ini, we have spaces after the comm
+
+[flake8]
+ignore = D1
[testenv]
-commands =
- py.test tests
- flake8 abp tests setup.py
deps =
pytest
mock
- flake8
- pep8-naming
- hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake8-eyeo
+commands =
+ py.test tests
Sebastian Noack 2017/10/11 20:28:51 In no other tox.ini, we list commands before deps.
[testenv:coverage]
-commands =
- py.test --cov=abp tests
deps =
pytest-cov
mock
+commands =
+ py.test --cov=abp tests
# We have to install our package in development mode
# so that pytest-cov can find the .coverage file.
usedevelop = true
+
+[testenv:flake8]
Sebastian Noack 2017/10/11 20:28:51 In all other tox.ini that have more than one teste
+basepython = python3
+skip_install = true
+deps =
+ flake8
+ flake8-docstrings
Sebastian Noack 2017/10/11 20:28:51 Using flake8-docstrings is now part of our coding
+ pep8-naming
+ hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake8-eyeo
+commands =
+ flake8
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld