Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Discussion: should we include Papaparse (methods) in codebase (20kb minified) #26

Closed
DataGridXL opened this issue Feb 8, 2022 · 1 comment

Comments

@DataGridXL
Copy link
Owner

We're using quite a bit of methods to parse and print tsv and csv. Papaparse is only 20kb minified, and it's heavily tested. They have solved edge cases that we have yet to encounter. Discuss!

@DataGridXL
Copy link
Owner Author

Yes. Included since 2.22.02.22. We have embedded a ES6ified version of papa parse in the source, which adds only 20kb minified. Worth it, so we can use parse and unparse methods. All the edge cases we still have to encounter, are now solved by a heavily tested and heavily used lib.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant