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

Issue 29377837: Issue 3692 - Remove curly ESLint rule (Closed)

Created:
March 3, 2017, 10:03 a.m. by kzar
Modified:
March 7, 2017, 10:19 a.m.
CC:
Felix Dahlke, wspee
Visibility:
Public.

Description

Issue 3692 - Remove curly ESLint rule See discussion https://codereview.adblockplus.org/29377779/

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M eslint-config-eyeo/index.js View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 3
kzar
Patch Set 1
March 3, 2017, 10:06 a.m. (2017-03-03 10:06:13 UTC) #1
Wladimir Palant
Looking at the configuration options of the curly rule, none of them really match our ...
March 5, 2017, 3:53 p.m. (2017-03-05 15:53:24 UTC) #2
Thomas Greiner
March 6, 2017, 12:02 p.m. (2017-03-06 12:02:48 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld