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

Issue 6308119894294528: Issue 2274 - Move notification parsing into a module (Closed)

Created:
April 4, 2015, 11:29 p.m. by Felix Dahlke
Modified:
May 22, 2015, 1:01 p.m.
Reviewers:
Sebastian Noack
CC:
Wladimir Palant
Visibility:
Public.

Description

Issue 2274 - Move notification parsing into a module

Patch Set 1 : #

Patch Set 2 : hg mv generateNotifications.py to parser.py #

Patch Set 3 : Don't set the version in parser.load_notifications() #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -109 lines) Patch
M sitescripts/management/bin/generateNotifications.py View 1 2 1 chunk +13 lines, -90 lines 0 comments Download
A sitescripts/notifications/__init__.py View 0 chunks +-1 lines, --1 lines 0 comments Download
M sitescripts/notifications/parser.py View 1 2 5 chunks +18 lines, -20 lines 0 comments Download

Messages

Total messages: 6
Felix Dahlke
Split #2162 up a bit, it got too big. Here's the first rather small chunk. ...
April 4, 2015, 11:43 p.m. (2015-04-04 23:43:07 UTC) #1
Sebastian Noack
I can't review this patch like that. Please use "hg mv".
April 5, 2015, 11:12 a.m. (2015-04-05 11:12:23 UTC) #2
Felix Dahlke
New patch set using hg mv is up. Note that I'm not actually moving generateNotifications.py, ...
April 5, 2015, 4:30 p.m. (2015-04-05 16:30:50 UTC) #3
Sebastian Noack
LGTM
April 5, 2015, 4:49 p.m. (2015-04-05 16:49:39 UTC) #4
Felix Dahlke
Minor update here: I moved setting the version out of load_notifications() - doesn't really belong ...
April 15, 2015, 9:25 a.m. (2015-04-15 09:25:53 UTC) #5
Sebastian Noack
April 15, 2015, 3:10 p.m. (2015-04-15 15:10:02 UTC) #6
Still LGTM

Powered by Google App Engine
This is Rietveld