Disable require.ensure() warning (#2177)

dadf93b2 · Dan Abramov · GitHub · 66792dde · dadf93b2
Commit dadf93b2 authored 7 years ago by Dan Abramov Committed by GitHub 7 years ago
Hide whitespace changes
Inline Side-by-side

Showing

with 7 additions and 5 deletions
+7 -5
--- a/packages/eslint-config-react-app/index.js
+++ b/packages/eslint-config-react-app/index.js
@@ -220,11 +220,13 @@ module.exports = {
    'valid-typeof': 'warn',
    'no-restricted-properties': [
      'error',
-      {
-        object: 'require',
-        property: 'ensure',
-        message: 'Please use import() instead. More info: https://webpack.js.org/guides/code-splitting-import/#dynamic-import',
-      },
+      // TODO: reenable once import() is no longer slow.
+      // https://github.com/facebookincubator/create-react-app/issues/2176
+      // {
+      //   object: 'require',
+      //   property: 'ensure',
+      //   message: 'Please use import() instead. More info: https://webpack.js.org/guides/code-splitting-import/#dynamic-import',
+      // },
      {
        object: 'System',
        property: 'import',