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

Issue 29512597: Issue 4491 - Adds handling for block commented page metadata (Closed)

Created:
Aug. 11, 2017, 8:36 a.m. by Jon Sonesen
Modified:
Aug. 15, 2017, 11 a.m.
Reviewers:
Vasily Kuznetsov
Visibility:
Public.

Description

Issue 4491 - Adds handling for block commented page metadata

Patch Set 1 #

Total comments: 2

Patch Set 2 : add strip call to line check #

Total comments: 1

Patch Set 3 : ;) #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M cms/converters.py View 1 2 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 5
Jon Sonesen
Aug. 11, 2017, 8:36 a.m. (2017-08-11 08:36:11 UTC) #1
Vasily Kuznetsov
Hi Jon, Looks pretty good. Just one suggestion: https://codereview.adblockplus.org/29512597/diff/29512598/cms/converters.py File cms/converters.py (right): https://codereview.adblockplus.org/29512597/diff/29512598/cms/converters.py#newcode125 cms/converters.py:125: if ...
Aug. 11, 2017, 5:38 p.m. (2017-08-11 17:38:22 UTC) #2
Jon Sonesen
Thanks for looking! How's this? https://codereview.adblockplus.org/29512597/diff/29512598/cms/converters.py File cms/converters.py (right): https://codereview.adblockplus.org/29512597/diff/29512598/cms/converters.py#newcode125 cms/converters.py:125: if line == '<!--\n' ...
Aug. 14, 2017, 5:58 a.m. (2017-08-14 05:58:52 UTC) #3
Vasily Kuznetsov
LGTM https://codereview.adblockplus.org/29512597/diff/29515555/cms/converters.py File cms/converters.py (right): https://codereview.adblockplus.org/29512597/diff/29515555/cms/converters.py#newcode125 cms/converters.py:125: if line.strip() == '<!--' or line.strip() == '-->': ...
Aug. 14, 2017, 11:20 a.m. (2017-08-14 11:20:07 UTC) #4
Jon Sonesen
Aug. 15, 2017, 10:45 a.m. (2017-08-15 10:45:25 UTC) #5
;)

Powered by Google App Engine
This is Rietveld