rstacruz / jsdom-global

Enable DOM in Node.js
477 stars 38 forks source link

Bad practice to set the global variables in Node.js #29

Open joetidee opened 7 years ago

joetidee commented 7 years ago

The JSDom documentation advises against making window and document , etc. part of the global namespace in Node.js. Should I therefore avoid using this package, or do you have a counter- argument?

saivan commented 6 years ago

Yes, I'm actually having issues with mocha randomly exiting during watch mode when this package is included. This seems dangerous to me.