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

Unified Diff: README.md

Issue 29808587: Issue 6717 - Part 1: run eslint through npm (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Use eslint5. Fix eslint error. Created July 13, 2018, 3:04 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « .hgignore ('k') | ext/background.js » ('j') | ext/background.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
===================================================================
--- a/README.md
+++ b/README.md
@@ -87,13 +87,13 @@
The unit tests will run automatically once the page loads.
Linting
-------
You can lint the code using [ESLint](http://eslint.org).
kzar 2018/08/08 11:15:19 Would you mind adding a link to our configuration
hub 2018/08/14 11:37:03 reworded a bit the whole paragraph and added the l
- eslint *.js lib/ qunit/ ext/ chrome/
+ npm run lint
-You will need to set up ESLint and our configuration first, see
-[eslint-config-eyeo](https://hg.adblockplus.org/codingtools/file/tip/eslint-config-eyeo)
-for more information.
+You will need to run at first
+
+ npm install
« no previous file with comments | « .hgignore ('k') | ext/background.js » ('j') | ext/background.js » ('J')

Powered by Google App Engine
This is Rietveld