Closed dontcallmedom closed 9 years ago
Guess we will need to update our tests before merging that
seems like jsdom < 4.x is not compatible. why did we need to downgrade jsdom?
jsdom refuses to run with node.js (only io.js) in versions >= 4.x
well, that's interesting. jsdom refuses for all node version? even 0.12 ?
https://github.com/tmpvar/jsdom says
Note that as of our 4.0.0 release, jsdom no longer works with Node.js™, and instead requires io.js. You are still welcome to install a release in the 3.x series if you use Node.js™.
any reason not to merge this?
the Travis build failed no?
But not due to this change specifically as far as I can tell. Our Travis CI set up seems in a bad shape, but I wouldn't put that in front of a (fairly simple) bug fix.
makes sens :)
ref https://github.com/w3c/Mobile-Checker/issues/47#issuecomment-114635340