Remove outdated ldjson-stream

anandthakker / doiuse

:bomb: Lint CSS for browser support against caniuse database.

MIT License

1.23k stars 50 forks source link

Remove outdated ldjson-stream #171

Closed onigoetz closed 10 months ago

onigoetz commented 11 months ago

Hi,

This PR removes ldjson-stream, which hasn't been updated in 9 years and brings some very outdated dependencies (such as through2 0.6.5)

I'm replacing it with no dependency since the feature itself can be done easily in modern Node

onigoetz commented 10 months ago

Hi @anandthakker are you interested in this PR?

RJWadley commented 10 months ago

you'd have better luck with @clshortfuse :)

clshortfuse commented 10 months ago

I've lost track of a couple things, but I'll put this on the milestone for sure.

onigoetz commented 10 months ago

awesome, thanks :)

clshortfuse commented 10 months ago

Looks good. The only difference is ldjson-stream uses json-stringify-safe which doesn't throw on circular references. I don't think we have any. This also slips in fixing package-json which I hadn't done.