patchconv/tox.ini - Issue 29569634: Noissue - Added flake8-docstrings (codingtools)

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Delta Between Two Patch Sets: patchconv/tox.ini

Issue 29569634: Noissue - Added flake8-docstrings (codingtools) (Closed)

Left Patch Set: Fixed inconsistent indentation, set basepython for flake8 env Created Oct. 10, 2017, 6:11 p.m.

Right Patch Set: Don't install patchconv in flake8 venv Created Oct. 11, 2017, 3:28 a.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

Left: Side by side diff | Download
Right: Side by side diff | Download

LEFT	RIGHT
1 [tox]	1 [tox]

2 envlist = py27,py35,py36,flake8	2 envlist = py27,py35,py36,flake8

3	3

4 [flake8]	4 [flake8]

5 ignore = D1	5 ignore = D1

6	6

7 [testenv]	7 [testenv]

8 deps =	8 deps =

9 pytest	9 pytest

10 pytest-cov	10 pytest-cov

11 mock	11 mock

12 commands =	12 commands =

13 py.test --cov=patchconv --cov-report term-missing tests	13 py.test --cov=patchconv --cov-report term-missing tests

14	14

15 [testenv:flake8]	15 [testenv:flake8]

16 basepython = python3	16 basepython = python3

	17 skip_install = true
	Sebastian Noack 2017/10/11 03:30:44 Since this environment is only for linting, there Since this environment is only for linting, there is no reason to install the package in here. (However, in case of flake8-eyeo it's different, since flake8-eyeo should be linted with flake8-eyeo.)
17 deps =	18 deps =

18 flake8	19 flake8

19 flake8-docstrings	20 flake8-docstrings

20 pep8-naming	21 pep8-naming

21 ../flake8-eyeo	22 ../flake8-eyeo

22 commands =	23 commands =

23 flake8 patchconv.py setup.py tests	24 flake8 patchconv.py setup.py tests

LEFT	RIGHT