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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Large dependency graph #221

Open
kachkaev opened this issue Sep 1, 2024 · 1 comment
Open

Large dependency graph #221

kachkaev opened this issue Sep 1, 2024 · 1 comment

Comments

@kachkaev
Copy link

kachkaev commented Sep 1, 2024

👋 folks! I’ve just tried manypkg instead of check-dependency-version-consistency – looks great!

There is a minor internal issue I would like to bring up. Installing @manypkg/[email protected] adds quite a lot of transient dependencies some of which are quite dated. An example would be [email protected] that has not been updated for six years.

Because of that, the lock file gets quite polluted. Here is my diff after swapping check-dependency-version-consistency with @manypkg/cli (quite a lot of new stuff):

Screenshot 2024-09-01 at 11 35 27

Because the new dependency graph is quite big and parts of it are dated, there is a risk of bumping into security advisories that will be hard to address. It’d be great if the number of deps could be made smaller and libraries like spawndamnit could be replaced with something else, if possible.

Despite this small concern, great tool folks! I really like the simplicity of the DX you’ve created!

@kachkaev kachkaev changed the title Alternatives to [email protected] dependency Large dependency graph Sep 1, 2024
@Andarist
Copy link
Collaborator

Andarist commented Sep 1, 2024

We could accept a PR swapping this dependency for a lighter alternative. I'm not sure what that alternative would be though.

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

2 participants