mholt/PapaParse

ES6 module for PapaParse

Open

#813 opened on Jun 30, 2020

View on GitHub
 (7 comments) (22 reactions) (0 assignees)JavaScript (11,877 stars) (1,157 forks)batch import
api changeenhancementhelp wanted

Description

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?

Contributor guide

Tech stack
javascript
Domain
developer experiencebuild system
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Understanding of ES6 modulesFamiliarity with package.json exportsKnowledge of CommonJS vs ESM
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
35
Research direction
The issue requests an ES6 module export for PapaParse to avoid Angular build warnings. Research the existing package.json to see if a 'module' field or 'exports' map can be added. Look at how other libraries like lodash or date fns provide ESM builds. Consider using a build tool like Rollup to generate an ES module version from the current CommonJS source. Check if there are any existing PRs or branches attempting this.
ES6 module for PapaParse · mholt/PapaParse#813 | Good First Issue