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

Issue 29756682: Noissue - Adapt best practices for trailing commas (codingtools) (Closed)

Created:
April 19, 2018, 1:05 p.m. by Sebastian Noack
Modified:
April 19, 2018, 4:21 p.m.
Visibility:
Public.

Description

Noissue - Adapt best practices for trailing commas (codingtools)

Patch Set 1 #

Patch Set 2 : Added flake8-commas extension #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -6 lines) Patch
M cms-dev/tests/test_cms_cmp.py View 2 chunks +2 lines, -2 lines 0 comments Download
M cms-dev/tox.ini View 1 1 chunk +1 line, -0 lines 0 comments Download
M flake8-eyeo/setup.py View 1 chunk +1 line, -1 line 0 comments Download
M flake8-eyeo/tox.ini View 1 1 chunk +1 line, -0 lines 0 comments Download
M patchconv/setup.py View 1 chunk +2 lines, -2 lines 0 comments Download
M patchconv/tests/test_rietveld_to_git.py View 1 chunk +1 line, -1 line 0 comments Download
M patchconv/tox.ini View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2
Sebastian Noack
see https://codereview.adblockplus.org/29756624/
April 19, 2018, 1:06 p.m. (2018-04-19 13:06:10 UTC) #1
Vasily Kuznetsov
April 19, 2018, 3:41 p.m. (2018-04-19 15:41:58 UTC) #2
On 2018/04/19 13:06:10, Sebastian Noack wrote:
> see https://codereview.adblockplus.org/29756624/

LGTM

Powered by Google App Engine
This is Rietveld