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

Issue 29544671: Issue 5509 - add ESLint config (Closed)

Created:
Sept. 14, 2017, 12:25 p.m. by sergei
Modified:
Sept. 15, 2017, 4:36 p.m.
Reviewers:
hub
CC:
Felix Dahlke
Base URL:
https://github.com/adblockplus/libadblockplus.git
Visibility:
Public.

Description

So far it's merely the prerequisites and the config and one has to run it manually. This step should be done first because otherwise it's difficult to guess what else changes are required.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -0 lines) Patch
A .eslintignore View 1 chunk +2 lines, -0 lines 0 comments Download
A .eslintrc.json View 1 chunk +4 lines, -0 lines 0 comments Download
M .gitignore View 1 chunk +1 line, -0 lines 0 comments Download
M .hgignore View 1 chunk +1 line, -0 lines 0 comments Download
M README.md View 1 chunk +11 lines, -0 lines 0 comments Download
A lib/.eslintrc.json View 1 chunk +14 lines, -0 lines 0 comments Download
A package.json View 1 chunk +13 lines, -0 lines 0 comments Download

Messages

Total messages: 5
sergei
Sept. 14, 2017, 12:28 p.m. (2017-09-14 12:28:10 UTC) #1
hub
LGTM
Sept. 14, 2017, 1:53 p.m. (2017-09-14 13:53:07 UTC) #2
hub
btw, do you want me to make the JS pass the linter?
Sept. 14, 2017, 5:20 p.m. (2017-09-14 17:20:34 UTC) #3
sergei
On 2017/09/14 17:20:34, hub wrote: > btw, do you want me to make the JS ...
Sept. 14, 2017, 6:46 p.m. (2017-09-14 18:46:25 UTC) #4
hub
Sept. 15, 2017, 1:37 p.m. (2017-09-15 13:37:17 UTC) #5

Powered by Google App Engine
This is Rietveld