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

Issue 29569627: Noissue - Added flake8-docstrings (buildtools) (Closed)

Created:
Oct. 9, 2017, 1:31 a.m. by Sebastian Noack
Modified:
Oct. 10, 2017, 4:59 a.m.
Visibility:
Public.

Description

Noissue - Added flake8-docstrings (buildtools)

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -5 lines) Patch
M tox.ini View 2 chunks +8 lines, -5 lines 1 comment Download

Messages

Total messages: 3
Sebastian Noack
https://codereview.adblockplus.org/29569627/diff/29569628/tox.ini File tox.ini (right): https://codereview.adblockplus.org/29569627/diff/29569628/tox.ini#newcode6 tox.ini:6: ignore = D1 D1 is for missing docstrings. I ...
Oct. 9, 2017, 1:57 a.m. (2017-10-09 01:57:22 UTC) #1
Vasily Kuznetsov
LGTM I like how because of flake8-per-file-ignores and its error X100 I don't worry that ...
Oct. 9, 2017, 8:42 p.m. (2017-10-09 20:42:37 UTC) #2
tlucas
Oct. 9, 2017, 11:24 p.m. (2017-10-09 23:24:39 UTC) #3
LGTM - Also looking forward to proper docstring linting :)

Powered by Google App Engine
This is Rietveld