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

Issue 6292212006518784: Noissue - Added documentation for exported functions to the "whitelisting" module (Closed)

Created:
Feb. 28, 2015, 5:14 p.m. by Sebastian Noack
Modified:
March 3, 2015, 8:29 a.m.
Reviewers:
kzar, Wladimir Palant
Visibility:
Public.

Description

Noissue - Added documentation for exported functions to the "whitelisting" module

Patch Set 1 : #

Patch Set 2 : Fixed typo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+34 lines, -0 lines) Patch
M lib/whitelisting.js View 1 4 chunks +34 lines, -0 lines 0 comments Download

Messages

Total messages: 5
Sebastian Noack
Feb. 28, 2015, 5:14 p.m. (2015-02-28 17:14:19 UTC) #1
kzar
LGTM
March 2, 2015, 5:05 p.m. (2015-03-02 17:05:50 UTC) #2
Sebastian Noack
Sorry, there was a typo. It's a response header of course.
March 2, 2015, 5:14 p.m. (2015-03-02 17:14:26 UTC) #3
kzar
Whoops, missed that. LGTM
March 2, 2015, 5:16 p.m. (2015-03-02 17:16:50 UTC) #4
Wladimir Palant
March 2, 2015, 7:14 p.m. (2015-03-02 19:14:40 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld