exceljs / exceljs

Excel Workbook Manager
MIT License
13.09k stars 1.67k forks source link

[BUG] dependencies should be moved to devDependencies #2762

Open devkral opened 2 weeks ago

devkral commented 2 weeks ago

🐛 Bug Report

Currently exceljs defines dependencies which should live in devDependencies. This way it bloats the installation by magnitudes while just the dist folder is required.

My workaround consists of using the overrides.

Lib version: 4.4.0

Steps To Reproduce

init a new project, add exceljs as dependency and initialize the project with npm install

Is:

The node_modules has many packages

Should be:

The node_modules has around 2 packages

So please reduce the bloat, move just the dependencies to the devDependencies.

And if you really care for package size, then just include the dist folder in the npm package