forked from cdklabs/aws-delivlib
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.eslintrc.json
132 lines (111 loc) · 4.56 KB
/
.eslintrc.json
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
{
"env": {
"jest": true,
"node": true
},
"plugins": [
"@typescript-eslint",
"import"
],
"parser": "@typescript-eslint/parser",
"parserOptions": {
"ecmaVersion": 2018,
"sourceType": "module",
"project": "./tsconfig.json"
},
"extends": [
"plugin:import/typescript"
],
"settings": {
"import/parsers": {
"@typescript-eslint/parser": [".ts", ".tsx"]
},
"import/resolver": {
"node": {},
"typescript": {
"directory": "./tsconfig.json"
}
}
},
"ignorePatterns": ["*.js", "*.d.ts", "node_modules/", "change-control-lambda/"],
"rules": {
// Require use of the `import { foo } from "bar";` form instead of `import foo = require("bar");`
"@typescript-eslint/no-require-imports": ["error"],
"@typescript-eslint/indent": ["error", 2],
// Style
"quotes": ["error", "single", { "avoidEscape": true }],
"comma-dangle": ["error", "always-multiline"], // ensures clean diffs, see https://medium.com/@nikgraf/why-you-should-enforce-dangling-commas-for-multiline-statements-d034c98e36f8
"comma-spacing": ["error", { "before": false, "after": true }], // space after, no space before
"no-multi-spaces": ["error", { "ignoreEOLComments": false }], // no multi spaces
"array-bracket-spacing": ["error", "never"], // [1, 2, 3]
"array-bracket-newline": ["error", "consistent"], // enforce consistent line breaks between brackets
"object-curly-spacing": ["error", "always"], // { key": "value" }
"object-curly-newline": ["error", { "multiline": true, "consistent": true }], // enforce consistent line breaks between braces
"object-property-newline": ["error", { "allowAllPropertiesOnSameLine": true }], // enforce "same line" or "multiple line" on object properties
"keyword-spacing": ["error"], // require a space before & after keywords
"brace-style": ["error", "1tbs", { "allowSingleLine": true }], // enforce one true brace style
"space-before-blocks": "error", // require space before blocks
"curly": ["error", "multi-line", "consistent"], // require curly braces for multiline control statements
// Require all imported libraries actually resolve (!!required for import/no-extraneous-dependencies to work!!)
"import/no-unresolved": ["error"],
// Require an ordering on all imports
"import/order": ["error", {
"groups": ["builtin", "external"],
"alphabetize": { "order": "asc", "caseInsensitive": true }
}],
// Cannot import from the same module twice
"no-duplicate-imports": ["error"],
// Cannot shadow names
"no-shadow": ["off"],
"@typescript-eslint/no-shadow": ["error"],
// Required spacing in property declarations (copied from TSLint, defaults are good)
"key-spacing": ["error"],
// Require semicolons
"semi": ["error", "always"],
// Don"t unnecessarily quote properties
"quote-props": ["error", "consistent-as-needed"],
// No multiple empty lines
"no-multiple-empty-lines": ["error"],
// Max line lengths
"max-len": ["error", {
"code": 150,
"ignoreUrls": true, // Most common reason to disable it
"ignoreStrings": true, // These are not fantastic but necessary for error messages
"ignoreTemplateLiterals": true,
"ignoreComments": true,
"ignoreRegExpLiterals": true
}],
// One of the easiest mistakes to make
"@typescript-eslint/no-floating-promises": ["error"],
// Make sure that inside try/catch blocks, promises are "return await"ed
// (must disable the base rule as it can report incorrect errors)
"no-return-await": "off",
"@typescript-eslint/return-await": "error",
// Don"t leave log statements littering the premises!
"no-console": ["error"],
// Useless diff results
"no-trailing-spaces": ["error"],
// Must use foo.bar instead of foo["bar"] if possible
"dot-notation": ["error"],
// Must use "import" statements (disabled because it doesn"t add a lot over no-require-imports)
// "@typescript-eslint/no-var-requires": ["error"],
// Are you sure | is not a typo for || ?
"no-bitwise": ["error"],
// Member ordering
"@typescript-eslint/member-ordering": ["error", {
"default": [
"public-static-field",
"public-static-method",
"protected-static-field",
"protected-static-method",
"private-static-field",
"private-static-method",
"field",
// Constructors
"constructor", // = ["public-constructor", "protected-constructor", "private-constructor"]
// Methods
"method"
]
}]
}
}