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

Issue 29923555: Noissue - Switch to binary operator after line break standard following PEP8 and Flake8 (Closed)

Created:
Oct. 24, 2018, 2:02 p.m. by Vasily Kuznetsov
Modified:
Oct. 24, 2018, 2:28 p.m.
Reviewers:
Tudor Avram
Base URL:
https://hg.adblockplus.org/cms
Visibility:
Public.

Description

Noissue - Switch to binary operator after line break standard following PEP8 and Flake8 Repository: https://hg.adblockplus.org/cms Base revision: db4d45b87795

Patch Set 1 #

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

Messages

Total messages: 2
Vasily Kuznetsov
Hi Tudor, It seems that Flake8 has started enforcing W504 (no line breaks after a ...
Oct. 24, 2018, 2:07 p.m. (2018-10-24 14:07:57 UTC) #1
Tudor Avram
Oct. 24, 2018, 2:28 p.m. (2018-10-24 14:28:00 UTC) #2
LGTM

Cool, thanks for taking care of this... Don't know how I missed this yesterday
when I ran the tests for the test server.

Powered by Google App Engine
This is Rietveld