Comment on Show HN: Npm module that makes easy to convert JSON to CSVComments−dc29yFor a <5Kb library, I recommend removing the underscore dependency. You should make this as lightweight as possible if you want a lot of adoption.−stdbrouw9yConverting to CSV is mainly a server-side problem, not something you'd want to do in the browser, so this doesn't really feel like an issue to me.−kauegimenesOP9yThanks for the tip, you are welcome to submit a PR if you want to =)−d2xdy29yNot OP, but I just submitted a PR removing underscore as a dep. :)−kauegimenesOP9yThats awesome! thanks!PR: https://github.com/kauegimenes/jsonexport/pull/27−kauegimenesOP9yThis is now avaliable in v2.0.7 =)DependenciesNone−JoshGlazebrook9yYeah this looks like it's only using helper functions like _.isFunction(), _.isNumber(), and _.extend().
Comments
For a <5Kb library, I recommend removing the underscore dependency. You should make this as lightweight as possible if you want a lot of adoption.
Converting to CSV is mainly a server-side problem, not something you'd want to do in the browser, so this doesn't really feel like an issue to me.
Thanks for the tip, you are welcome to submit a PR if you want to =)
Not OP, but I just submitted a PR removing underscore as a dep. :)
Thats awesome! thanks!
PR: https://github.com/kauegimenes/jsonexport/pull/27
This is now avaliable in v2.0.7 =)
Dependencies
None
Yeah this looks like it's only using helper functions like _.isFunction(), _.isNumber(), and _.extend().