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.

Sign up for GitHub

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

Jump to bottom

common: remove csvtojson #1020

Open
josephjclark opened this issue Feb 24, 2025 · 0 comments
Open

common: remove csvtojson #1020

josephjclark opened this issue Feb 24, 2025 · 0 comments
Labels
bug Something isn't working Needs Priority This issue needs to be prioritised

Comments

@josephjclark
Copy link
Collaborator

It looks like common has an unused dependency on csvtojson. We use csv-parse instead, but csvtojson is still downloaded. And yet it seems to add 8mb to the npm install footprint, which slows down runtime.

We should remove it
@josephjclark josephjclark added bug Something isn't working Needs Priority This issue needs to be prioritised labels Feb 24, 2025
@github-project-automation github-project-automation bot moved this to New Issues in v2 Feb 24, 2025
@josephjclark josephjclark mentioned this issue Feb 24, 2025
Open
@theroinaochieng theroinaochieng moved this from New Issues to DevX Backlog in v2 Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working Needs Priority This issue needs to be prioritised
Projects
v2
Status: DevX Backlog
Milestone
No milestone
Development

No branches or pull requests
1 participant
@josephjclark