Can't run tests on Mac about sunder HOT 5 CLOSED

npm rebuild and try again? It's an electron version mismatch.

from sunder.

@GabeIsman npm rebuild did indeed fix the issue. Should we document this in the README somewhere as a potentially helpful troubleshooting step for other developers? I'm not sure how that would be best worded—what was about it that circumstance that made you think "npm rebuild" will fix this? I had already tried an rm -rf node_modules && npm install, so I'm slightly confused as to why that didn't work, but npm rebuild did.

Sorry for being such an Electron/Node n00b 😝 .

from sunder.

Any time I see a 'version mismatch' or 'binding not found' error I assume that I need to rebuild the native deps with npm rebuild.

I think we should probably be running npm rebuild in the postinstall hook, so it gets run every time you npm install. It's smart enough to not rebuild deps that don't need it, so shouldn't add too much time.

I think that npm rebuild is a fairly recent addition to the npm universe, although I'm not sure. Put this in the category of things about the development setup that should be updated to match the latest in the blindingly quick succession of JS development best practices.

Let alone that Webpack now has a completely new, non-backwards-compatible API. 🤦‍♂️

from sunder.

I would very much like to pin a release channel, e.g. LTS, and use that across all platforms. Then we can declare one and only one version of NodeJS in the README (since Mac users must download separately).

from sunder.

I think the root cause of this will be resolved with #38, leaving aside the issue of node versions.

from sunder.