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.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] dependencies should be moved to devDependencies #2762

Open
devkral opened this issue May 14, 2024 · 0 comments
Open

[BUG] dependencies should be moved to devDependencies #2762

devkral opened this issue May 14, 2024 · 0 comments

Comments

@devkral
Copy link

devkral commented May 14, 2024

馃悰 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant