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

Issue 29569623: Noissue - Added flake8-docstrings (cms) (Closed)

Created:
Oct. 9, 2017, 1:30 a.m. by Sebastian Noack
Modified:
Oct. 10, 2017, 4:59 a.m.
Reviewers:
Vasily Kuznetsov
CC:
tlucas, Jon Sonesen
Visibility:
Public.

Description

Noissue - Added flake8-docstrings (cms)

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -3 lines) Patch
M tests/test_translations.py View 1 chunk +1 line, -2 lines 1 comment Download
M tox.ini View 2 chunks +4 lines, -1 line 1 comment Download

Messages

Total messages: 2
Sebastian Noack
https://codereview.adblockplus.org/29569623/diff/29569624/tests/test_translations.py File tests/test_translations.py (right): https://codereview.adblockplus.org/29569623/diff/29569624/tests/test_translations.py#newcode49 tests/test_translations.py:49: for (url, data), (expect_url, expect_data) in zip(app.request_log, For some ...
Oct. 9, 2017, 1:59 a.m. (2017-10-09 01:59:12 UTC) #1
Vasily Kuznetsov
Oct. 9, 2017, 8:43 p.m. (2017-10-09 20:43:39 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld