Left: | ||
Right: |
LEFT | RIGHT |
---|---|
1 # eslint-config-eyeo | 1 # eslint-config-eyeo |
2 | 2 |
3 An [ESLint](http://eslint.org) configuration that checks for compliance with the | 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) | 4 [Adblock Plus coding style guide](https://adblockplus.org/coding-style#javascrip t) |
5 which is used for all eyeo projects. | 5 which is used for all eyeo projects. |
6 | 6 |
7 ## Installation | 7 ## Installation |
8 | 8 |
9 npm install -g eslint eslint-config-eyeo | 9 npm install -g eslint eslint-config-eyeo |
10 | 10 |
11 (As root, or using sudo.) | 11 This command requires administrator privileges so you might need to use `sudo`. |
Wladimir Palant
2017/02/14 12:43:20
Nit: I'm not a big fan of having an entire sentenc
kzar
2017/02/15 05:24:22
(Done.)
| |
12 | 12 |
13 ## Usage | 13 ## Usage |
14 | 14 |
15 To lint a JavaScript file using ESLint you simply run the `eslint` command with | 15 To lint a JavaScript file using ESLint you run the `eslint` command with the |
Wladimir Palant
2017/02/14 12:43:20
Nit: Please avoid using the word "simply" in instr
kzar
2017/02/15 05:24:22
Good point, Done.
| |
16 the file as an argument. For example: | 16 file as an argument. For example: |
17 | 17 |
18 eslint some-file.js | 18 eslint some-file.js |
19 | 19 |
20 (For advanced usage see `eslint --help`.) | 20 For advanced usage see `eslint --help`. |
Wladimir Palant
2017/02/14 12:43:20
Nit: Parentheses are unnecessary here.
kzar
2017/02/15 05:24:22
Done.
| |
21 | 21 |
22 In order to use eslint-config-eyeo your project's ESLint configuration | 22 In order to use eslint-config-eyeo your project's ESLint configuration |
23 should extend from it. A minimal example looks like this: | 23 should extend from it. A minimal example looks like this: |
24 | 24 |
25 { | 25 { |
26 "extends": "eslint-config-eyeo", | 26 "extends": "eslint-config-eyeo", |
27 "root": true | 27 "root": true |
28 } | 28 } |
29 | 29 |
30 (For projects without an ESLint configuration you can create your own | 30 For projects without an ESLint configuration you can create your own |
31 personal configuration in `~/.eslintrc.json`, but take care to remove | 31 personal configuration in `~/.eslintrc.json`, but take care to remove |
32 the `"root": true` section from the above example.) | 32 the `"root": true` section from the above example. |
Wladimir Palant
2017/02/14 12:43:20
Nit: Parentheses are unnecessary here.
kzar
2017/02/15 05:24:22
Done.
| |
LEFT | RIGHT |