@@ -109,16 +109,6 @@ module.exports = {
109
109
} ,
110
110
} ,
111
111
112
- settings : {
113
- 'import/ignore' : [ 'node_modules' ] ,
114
- 'import/extensions' : [ '.js' ] ,
115
- 'import/resolver' : {
116
- node : {
117
- extensions : [ '.js' , '.json' ] ,
118
- } ,
119
- } ,
120
- } ,
121
-
122
112
rules : {
123
113
// http://eslint.org/docs/rules/
124
114
'array-callback-return' : 'warn' ,
@@ -242,38 +232,13 @@ module.exports = {
242
232
} ,
243
233
] ,
244
234
245
- // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/
246
-
247
- // TODO: import rules are temporarily disabled because they don't play well
248
- // with how eslint-loader only checks the file you change. So if module A
249
- // imports module B, and B is missing a default export, the linter will
250
- // record this as an issue in module A. Now if you fix module B, the linter
251
- // will not be aware that it needs to re-lint A as well, so the error
252
- // will stay until the next restart, which is really confusing.
253
-
254
- // This is probably fixable with a patch to eslint-loader.
255
- // When file A is saved, we want to invalidate all files that import it
256
- // *and* that currently have lint errors. This should fix the problem.
257
- // (As an exception, import/no-webpack-loader-syntax can be enabled already
258
- // because it doesn't depend on whether the file exists, so this issue
259
- // doesn't apply to it.)
260
-
261
- // 'import/default': 'warn',
262
- // 'import/export': 'warn',
263
- // 'import/named': 'warn',
264
- // 'import/namespace': 'warn',
265
- // 'import/no-amd': 'warn',
266
- // 'import/no-duplicates': 'warn',
267
- // 'import/no-extraneous-dependencies': 'warn',
268
- // 'import/no-named-as-default': 'warn',
269
- // 'import/no-named-as-default-member': 'warn',
270
- // 'import/no-unresolved': ['warn', { commonjs: true }],
271
- // We don't support configuring Webpack using import source strings, so this
272
- // is always an error.
235
+ // https://github.com/benmosher/eslint-plugin-import/tree/master/docs/rules
273
236
'import/no-webpack-loader-syntax' : 'error' ,
274
237
275
238
// https://github.com/yannickcr/eslint-plugin-react/tree/master/docs/rules
239
+ 'react/jsx-no-comment-textnodes' : 'warn' ,
276
240
'react/jsx-no-duplicate-props' : [ 'warn' , { ignoreCase : true } ] ,
241
+ 'react/jsx-no-target-blank' : 'warn' ,
277
242
'react/jsx-no-undef' : 'error' ,
278
243
'react/jsx-pascal-case' : [
279
244
'warn' ,
@@ -289,7 +254,7 @@ module.exports = {
289
254
'react/no-direct-mutation-state' : 'warn' ,
290
255
'react/no-is-mounted' : 'warn' ,
291
256
'react/react-in-jsx-scope' : 'error' ,
292
- 'react/require-render-return' : 'warn ' ,
257
+ 'react/require-render-return' : 'error ' ,
293
258
'react/style-prop-object' : 'warn' ,
294
259
295
260
// https://github.com/evcohen/eslint-plugin-jsx-a11y/tree/master/docs/rules
0 commit comments