Support for older node versions

[kunalspathak]

Removed condition that was prohibiting NVS using for node versions older than 0.10. Bumped the version to 1.3.1.

[jasongin]

What older versions did you test with?

[kunalspathak]

I tested with 0.7, 0.8 and 0.9. It doesn't work with 0.1, 0.2, 0.3, 0.4, 0.5 because of missing files in index.json. Didn't bother finding why they failed.

[ljharb]

What about 0.6? 0.7 and 0.9 and 0.3 and 0.5 and 0.1 aren't important because they're unstable by the apache model, but 0.6 and 0.8 are stable.

Also 0.4 would be nice but isn't quite as important.

[kunalspathak]

We could support older versions if needed, but this PR is specially to unblock expressjs/body-parser#233

[ljharb]

Sounds like the version check shouldn't be removed then; it should just be lowered to allow 0.7?

[kunalspathak]

Reasonable ask. Updated the PR.

[ljharb]

Seems legit 👍

it'd still be good to get things working on 0.4 and 0.6 tho :-)

[ljharb]

altho actually, this is semver-minor imo, not patch, and generally version numbers should never be bumped in PRs

[jasongin]

I suppose in a project that is often cherry-picking changes across multiple major-version branches/releases, then you wouldn't want the changes to include version numbers. But this project isn't that complex, so it doesn't matter to me.

As for semver-minor vs patch, I think it could go either way. So, this is fine.

[jasongin]

We could support older versions if needed, but this PR is specially to unblock expressjs/body-parser#233

That only goes down to version 0.8. So why go to 0.7 here (and not 0.6)?

[kunalspathak]

So why go to 0.7 here (and not 0.6)?

Not particular reason. I just found out that upto 0.7 gets supported with minimal changes needed. (just regex change). Supporting older version would need changes at multiple places and hence didn't add support for them.