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

Issue 29345200: Noissue - Added flake8/tox configuration to the CMS (Closed)

Created:
May 28, 2016, 12:45 p.m. by Sebastian Noack
Modified:
May 30, 2016, 12:17 p.m.
Reviewers:
Vasily Kuznetsov
CC:
Jon Sonesen
Visibility:
Public.

Description

Noissue - Added flake8/tox configuration to the CMS

Patch Set 1 #

Patch Set 2 : Run flake8 on runserver.spec #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -0 lines) Patch
M .gitignore View 1 chunk +1 line, -0 lines 0 comments Download
M .hgignore View 1 chunk +1 line, -0 lines 0 comments Download
A tox.ini View 1 1 chunk +23 lines, -0 lines 1 comment Download

Messages

Total messages: 3
Sebastian Noack
May 28, 2016, 12:45 p.m. (2016-05-28 12:45:24 UTC) #1
Sebastian Noack
https://codereview.adblockplus.org/29345200/diff/29345238/tox.ini File tox.ini (right): https://codereview.adblockplus.org/29345200/diff/29345238/tox.ini#newcode23 tox.ini:23: flake8 --ignore E501,F821,N802 runserver.spec flake8-putty doesn't consider files with ...
May 29, 2016, 10:19 a.m. (2016-05-29 10:19:04 UTC) #2
Vasily Kuznetsov
May 30, 2016, 11:19 a.m. (2016-05-30 11:19:11 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld