mholt/PapaParse

ES6 module for PapaParse

Open

#813 opened on Jun 30, 2020

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

Repository metrics

Stars
 (11,877 stars)
PR merge metrics
 (Avg merge 18h) (1 merged PR in 30d)

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

Research direction
Add an ES6 module export to PapaParse's source to allow tree shaking and avoid CommonJS warnings in Angular.
Tech stack
javascripttypescript
Domain
frontend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
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.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
JavaScriptES6 modules
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65