Delta Between Two Patch Sets: eslintrc.json
Issue 29346539:
Issue 3692 - Created global eslintrc (Closed)
Left Patch Set: Merged browser and plugin configs. Changed error to warn for all rules related to code style. Changed no-console and no-unused-var to warn.
Created June 24, 2016, 10:11 p.m.
Right Patch Set: turned no-eq-null and no-unused-vars off
Created Oct. 13, 2016, 2:13 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with change/comment
|
« no previous file
|
no next file »
|
no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| LEFT | RIGHT |
|---|---|
| (no file at all) | |
| 1 { | |
| 2 "extends": "eslint:recommended", | |
| 3 "rules": { | |
| 4 | |
| 5 /* MDN General practices */ | |
| 6 "no-else-return": "error", | |
| 7 "no-alert": "error", | |
| 8 "indent": ["error", 2], | |
| 9 "valid-jsdoc": "error", | |
| 10 "eol-last": "error", | |
| 11 "vars-on-top": "off", | |
|
Sebastian Noack
2017/02/01 19:50:34
According to the documentation, it seems vars-on-t
| |
| 12 | |
| 13 /* MDN Naming and formatting code */ | |
| 14 "no-trailing-spaces": "error", | |
| 15 "max-len": ["error", 80], | |
| 16 "brace-style": ["error", "allman"], | |
| 17 | |
| 18 /* MDN JavaScript practices */ | |
| 19 "consistent-return": "error", | |
| 20 "no-array-constructor": "error", | |
| 21 "no-new-object": "error", | |
| 22 | |
| 23 /* ABP General */ | |
| 24 "no-warning-comments": "error", | |
| 25 | |
| 26 /* Palant recommends */ | |
| 27 "no-unused-vars": "off", | |
|
Sebastian Noack
2017/02/01 19:50:34
I just noticed, instead of completely disabling no
| |
| 28 "array-bracket-spacing": "error", | |
| 29 "arrow-spacing": "error", | |
| 30 "comma-spacing": "error", | |
| 31 "comma-style": "error", | |
| 32 "computed-property-spacing": "error", | |
| 33 "key-spacing": "error", | |
| 34 "keyword-spacing": "error", | |
| 35 "new-parens": "error", | |
| 36 "func-call-spacing": "error", | |
| 37 "no-var": "error", | |
| 38 "no-whitespace-before-property": "error", | |
| 39 "object-curly-spacing": "error", | |
| 40 "operator-assignment": "error", | |
| 41 "operator-linebreak": "error", | |
| 42 "padded-blocks": ["error", "never"], | |
| 43 "prefer-rest-params": "error", | |
| 44 "prefer-spread": "error", | |
| 45 "require-yield": "error", | |
|
Sebastian Noack
2017/02/01 19:50:34
It seems this is already included in eslint:recomm
| |
| 46 "quotes": ["error", "double", {"avoidEscape": true}], | |
| 47 "semi": "error", | |
| 48 "semi-spacing": "error", | |
| 49 "space-before-function-paren": ["error", "never"], | |
| 50 "space-in-parens": "error", | |
| 51 "space-infix-ops": "error", | |
| 52 "space-unary-ops": "error", | |
| 53 "spaced-comment": "error", | |
| 54 "yield-star-spacing": "error" | |
| 55 } | |
| 56 } | |
| LEFT | RIGHT |
« no previous file
|
no next file »
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
