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
ES6 module for PapaParse #813
Comments
Currently the typescript defintions are managed here https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/papaparse So you should update them there. See #545 |
@pokoli That doesn't seem very relevant at all. ES6 modules are part of modern javascript, they have nothing to do with typescript. The TLDR; Having typescript definitions is not enough. We want ES6 modules. |
@PooSham Thanks for the explanation. I will reopen the issue. I think this should be part of a new major release to allow users continue using the current version. I will be happy to review any PR implementing ES6 modules. |
@pokoli Thank you for reopening it :) Yes, making it a part of a major release seems reasonable. I have quite a lot at work right now, but I might look at it in 2 weeks or so (can't promise though). |
Looking forward to this release. I fear my Angular app might break if we decide to continue upgrading to Angular 11. |
@zgosalvez Just upgraded to angular 11, no issues other than the library cannot be optimized for size. |
As a first step I've created a PR that introduces support in (hopefully) the least backwards incompatible way. The code I'm confident with, build system support less so. Anyone up for some integration testing? PR is #875 |
While upgrading to Angular 10, using PapaParse gives following warning message - WARNING in C:\parser.component.ts depends on papaparse. CommonJS or AMD dependencies can cause optimization bailouts.
This seems to be because PapaParse is CommonJS module dependency.
I am importing papaparse as follows -
import * as csvParser from 'papaparse';
and using it as -
csvParser.parse(file, {});
Can you help me with an ES6 module that I could use instead?
The text was updated successfully, but these errors were encountered: