deps: replace json-parse-helpfulerror with parse-json

[zkat]

I've replaced this with json-parse-better-errors, which is MIt-licensed. (previously, this was switching to parse-json, but that inlined jju, so no, we can't use it).

[coveralls]

Coverage remained the same at 70.87% when pulling 1ecef48 on zkat/parse-json-replacement into e44f800 on master.

[coveralls]

Coverage remained the same at 70.87% when pulling 1ecef48 on zkat/parse-json-replacement into e44f800 on master.

[isaacs]

As a profiteering capitalist overlord of "Open Source", this LGTM. 👍

[iarna]

parse-json inlines jju and thus still has WTFPL licencsed code:

https://github.com/sindresorhus/parse-json/blob/master/vendor/parse.js#L4

[MylesBorins]

I wonder if that invalidates the MIT license of the parse-json... this kind of ambiguity probably makes the PCOoOSS's (profiteering corporate overlords of open source software) nervous

¯\_(ツ)_/¯

Looks like you could lift the code from https://github.com/npm/node-fetch-npm/pull/6/files to fix this... or make a 📦 or something 😇

Thanks for the hard work!!!

[zkat]

@MylesBorins done

[coveralls]

Coverage remained the same at 70.87% when pulling 8e8e61a on zkat/parse-json-replacement into e44f800 on master.

[iarna]

LGTM

[iarna]

Actually, tests are failing, but after they're fixed, LGTM.

[coveralls]

Coverage remained the same at 70.87% when pulling e74e0bd on zkat/parse-json-replacement into e44f800 on master.

[coveralls]

Coverage remained the same at 70.87% when pulling e74e0bd on zkat/parse-json-replacement into e44f800 on master.