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

Issue 29335753: Issue 2598 - Add a README (adblockpluscore) (Closed)

Created:
Feb. 4, 2016, 7:22 p.m. by kzar
Modified:
Feb. 4, 2016, 8:14 p.m.
Visibility:
Public.

Description

Issue 2598 - Add a README

Patch Set 1 #

Total comments: 2

Patch Set 2 : Use alternative syntax for header #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -0 lines) Patch
A README.md View 1 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 5
kzar
Patch Set 1 (I couldn't think of much to say, open to suggestions though.)
Feb. 4, 2016, 7:23 p.m. (2016-02-04 19:23:08 UTC) #1
Felix Dahlke
Looks fine except for one nit, but please make Wladimir reviewer - module owner and ...
Feb. 4, 2016, 7:27 p.m. (2016-02-04 19:27:41 UTC) #2
Wladimir Palant
LGTM
Feb. 4, 2016, 7:33 p.m. (2016-02-04 19:33:33 UTC) #3
kzar
Patch Set 2 : Use alternative syntax for header Whoops, not sure why I only ...
Feb. 4, 2016, 7:34 p.m. (2016-02-04 19:34:04 UTC) #4
Felix Dahlke
Feb. 4, 2016, 8:14 p.m. (2016-02-04 20:14:36 UTC) #5
Message was sent while issue was closed.
LGTM

Powered by Google App Engine
This is Rietveld