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

Side by Side Diff: eslint-config-eyeo/README.md

Issue 29374555: Issue 3692 - Add base ESLint configuration (Closed)
Patch Set: Allow empty catch blocks Created Feb. 13, 2017, 9:21 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | eslint-config-eyeo/index.js » ('j') | eslint-config-eyeo/index.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 # eslint-config-eyeo
2
3 An [ESLint](http://eslint.org) configuration that checks for compliance with the
4 [Adblock Plus coding style guide](https://adblockplus.org/coding-style#javascrip t)
5 which is used for all eyeo projects.
6
7 ## Installation
8
9 npm install -g eslint eslint-config-eyeo
10
11 (As root, or using sudo.)
12
13 ## Usage
14
15 To lint a JavaScript file using ESLint you simply run the `eslint` command with
16 the file as an argument. For example:
17
18 eslint some-file.js
19
20 (For advanced usage see `eslint --help`.)
21
22 In order to use eslint-config-eyeo your project's ESLint configuration
23 should extend from it. A minimal example looks like this:
24
25 {
26 "extends": "eslint-config-eyeo",
27 "root": true
Sebastian Noack 2017/02/13 11:31:29 One could argue, documenting the root option here
kzar 2017/02/13 13:02:22 Well most projects will need more in their configu
28 }
29
30 (For projects without an ESLint configuration you can create your own
31 personal configuration in `~/.eslintrc.json`, but take care to remove
32 the `"root": true` section from the above example.)
OLDNEW
« no previous file with comments | « no previous file | eslint-config-eyeo/index.js » ('j') | eslint-config-eyeo/index.js » ('J')

Powered by Google App Engine
This is Rietveld