Blacklist confusing global variables (facebook#2130)

sidoshi · gaearon · commit 81196d1f2076 · 2017-05-14T19:47:04.000+01:00
* Blacklist confusing global variables

* Blacklist globals

* Fix kitchensink test to lint

* Edit the list

* Use no-restricted globals for this
diff --git a/fixtures/kitchensink/src/App.js b/fixtures/kitchensink/src/App.js
@@ -54,7 +54,7 @@ class App extends Component {
   }
 
   componentDidMount() {
-    const feature = location.hash.slice(1);
+    const feature = window.location.hash.slice(1);
     switch (feature) {
       case 'array-destructuring':
         import(

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ class App extends Component {`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`componentDidMount() {`
`57`		`- const feature = location.hash.slice(1);`
	`57`	`+ const feature = window.location.hash.slice(1);`
`58`	`58`	`switch (feature) {`
`59`	`59`	`case 'array-destructuring':`
`60`	`60`	`import(`