Revert "Forbid using window properties as global variables" (#2099)

* Revert "Ignore the yarn cache directory (#2063)" This reverts commit 94149c7. * Revert "Forbid using window properties as global variables (#1840)" This reverts commit 5a7d8e8.
facebook · May 8, 2017 · 3a527f2 · 3a527f2
1 parent 94149c7
commit 3a527f2
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 11 deletions.
diff --git a/packages/eslint-config-react-app/index.js b/packages/eslint-config-react-app/index.js
@@ -26,19 +26,13 @@ module.exports = {
   plugins: ['import', 'flowtype', 'jsx-a11y', 'react'],
 
   env: {
+    browser: true,
     commonjs: true,
     es6: true,
     jest: true,
     node: true,
   },
 
-  globals: {
-    document: true,
-    window: true,
-    console: true,
-    navigator: true
-  },
-
   parserOptions: {
     ecmaVersion: 6,
     sourceType: 'module',

diff --git a/packages/react-scripts/fixtures/kitchensink/src/App.js b/packages/react-scripts/fixtures/kitchensink/src/App.js
@@ -26,7 +26,7 @@ class BuiltEmitter extends Component {
   }
 
   handleReady() {
-    document.dispatchEvent(new window.Event('ReactFeatureDidMount'));
+    document.dispatchEvent(new Event('ReactFeatureDidMount'));
   }
 
   render() {
@@ -54,7 +54,7 @@ class App extends Component {
   }
 
   componentDidMount() {
-    const feature = window.location.hash.slice(1);
+    const feature = location.hash.slice(1);
     switch (feature) {
       case 'array-destructuring':
         import(

diff --git a/packages/react-scripts/fixtures/kitchensink/src/features/webpack/NoExtInclusion.js b/packages/react-scripts/fixtures/kitchensink/src/features/webpack/NoExtInclusion.js
@@ -11,7 +11,7 @@ import React from 'react';
 import aFileWithoutExt from './assets/aFileWithoutExt';
 
 const text = aFileWithoutExt.includes('base64')
-  ? window.atob(aFileWithoutExt.split('base64,')[1]).trim()
+  ? atob(aFileWithoutExt.split('base64,')[1]).trim()
   : aFileWithoutExt;
 
 export default () => (

diff --git a/packages/react-scripts/fixtures/kitchensink/src/features/webpack/UnknownExtInclusion.js b/packages/react-scripts/fixtures/kitchensink/src/features/webpack/UnknownExtInclusion.js
@@ -11,7 +11,7 @@ import React from 'react';
 import aFileWithExtUnknown from './assets/aFileWithExt.unknown';
 
 const text = aFileWithExtUnknown.includes('base64')
-  ? window.atob(aFileWithExtUnknown.split('base64,')[1]).trim()
+  ? atob(aFileWithExtUnknown.split('base64,')[1]).trim()
   : aFileWithExtUnknown;
 
 export default () => (