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

Issue 4800587984011264: Noissue - [cms] Cleanup imports (Closed)

Created:
May 5, 2015, 2:03 p.m. by Sebastian Noack
Modified:
May 5, 2015, 2:58 p.m.
Reviewers:
kzar, Wladimir Palant
Visibility:
Public.

Description

For consistency with our other code, and compliance with PEP-8: * Only one module per import statement * Empty line between corelib, own and third-party imports * Removed relative imports

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -9 lines) Patch
M cms/bin/generate_static_pages.py View 1 chunk +9 lines, -3 lines 0 comments Download
M cms/bin/test_server.py View 1 chunk +3 lines, -3 lines 0 comments Download
M cms/sources.py View 1 chunk +1 line, -1 line 0 comments Download
M cms/utils.py View 1 chunk +1 line, -1 line 0 comments Download
M runserver.py View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 2
Sebastian Noack
May 5, 2015, 2:06 p.m. (2015-05-05 14:06:06 UTC) #1
Wladimir Palant
May 5, 2015, 2:16 p.m. (2015-05-05 14:16:25 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld