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

Issue 5100557458997248: Noissue - Updated website repositories on modules page (Closed)

Created:
May 21, 2015, 7:52 a.m. by Sebastian Noack
Modified:
May 21, 2015, 8:19 a.m.
Reviewers:
Felix Dahlke, saroyanm
CC:
Thomas Greiner
Visibility:
Public.

Description

Noissue - Updated website repositories on modules page

Patch Set 1 #

Patch Set 2 : Added some more websites #

Total comments: 2

Patch Set 3 : Removed blank line #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -2 lines) Patch
M pages/modules.html View 1 2 1 chunk +14 lines, -2 lines 0 comments Download

Messages

Total messages: 4
Sebastian Noack
May 21, 2015, 7:52 a.m. (2015-05-21 07:52:37 UTC) #1
Sebastian Noack
May 21, 2015, 8:08 a.m. (2015-05-21 08:08:32 UTC) #2
saroyanm
LGTM with small consistency fix Nit http://codereview.adblockplus.org/5100557458997248/diff/5724160613416960/pages/modules.html File pages/modules.html (right): http://codereview.adblockplus.org/5100557458997248/diff/5724160613416960/pages/modules.html#newcode432 pages/modules.html:432: Nit: Please remove ...
May 21, 2015, 8:17 a.m. (2015-05-21 08:17:05 UTC) #3
Sebastian Noack
May 21, 2015, 8:19 a.m. (2015-05-21 08:19:12 UTC) #4
http://codereview.adblockplus.org/5100557458997248/diff/5724160613416960/page...
File pages/modules.html (right):

http://codereview.adblockplus.org/5100557458997248/diff/5724160613416960/page...
pages/modules.html:432: 
On 2015/05/21 08:17:05, saroyanm wrote:
> Nit: Please remove this empty line for consistency

Done.

Powered by Google App Engine
This is Rietveld