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

Issue 29377994: Noissue - Remove confusing JavaScript brace rule (Closed)

Created:
March 7, 2017, 10:05 a.m. by kzar
Modified:
March 7, 2017, 11:57 a.m.
Visibility:
Public.

Description

Noissue - Remove confusing JavaScript brace rule As discussed in IRC.

Patch Set 1 #

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

Messages

Total messages: 4
kzar
Patch Set 1
March 7, 2017, 10:07 a.m. (2017-03-07 10:07:22 UTC) #1
Wladimir Palant
LGTM
March 7, 2017, 10:23 a.m. (2017-03-07 10:23:00 UTC) #2
Felix Dahlke
LGTM
March 7, 2017, 10:35 a.m. (2017-03-07 10:35:31 UTC) #3
Thomas Greiner
March 7, 2017, 11:57 a.m. (2017-03-07 11:57:48 UTC) #4
Message was sent while issue was closed.
LGTM

Powered by Google App Engine
This is Rietveld