make it clear that mathsteps requires node 6.0.0 about mathsteps

[Raibaz]

Nice one @emars :)

[emars]

If this is a recurring issue, something like:

http://stackoverflow.com/a/41620850/2816843

could be implemented.

Having a preinstall check that verifies that the user is on the proper version of node.

[evykassirer]

oh, sweet!

that looks very helpful and I totally think we should add it, thanks :)

[evykassirer]

script done! thanks to @Raibaz and to @emars for the suggestion

[evykassirer]

there's an issue where if you npm install mathsteps there's an error Error: Cannot find module 'semver'

I don't have time to look into it now, but hopefully we can fix it soon.

@aelnaiem ?

[Raibaz]

I think this is because preinstall is run before dependencies are installed...perhaps moving npm run requirements-check to install or postinstall would fix it?

[evykassirer]

yeah I think that's the issue too.. it's odd because both the stackoverflow answer and medium post user preinstall but then dependencies aren't installed yet... 😛 so maybe they're wrong! They don't have many votes, so that's possible.

I'm also looking around to try to find ways to test installing it again without patching a new version. Let me know if you know of anything

[aelnaiem]

It seems that node-semver should be installed with npm according to this:
https://docs.npmjs.com/files/package.json

But we still need to install it ourselves for this to work. Running it after the install seems to defeat the purpose. Going to continue looking into this

[aelnaiem]

Enforcing the engine was deprecated as part of an effort to shift control from module authors to application authors. Curious if this suggests that the best practice is to just notify application authors of the node version requirement in the README, and they can also see the requirement in the package.json

[evykassirer]

Sounds OK to me - too bad though :(

[evykassirer]

should be fixed now :)