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

Package "inflight" is deprecated & leaks memory #33

Open
ynot01 opened this issue Jun 7, 2024 · 2 comments
Open

Package "inflight" is deprecated & leaks memory #33

ynot01 opened this issue Jun 7, 2024 · 2 comments

Comments

@ynot01
Copy link

ynot01 commented Jun 7, 2024

From npm:
npm warn deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.

@Lauren1066
Copy link
Collaborator

Thank you for bringing this up!
I have determined mocha requires glob, which uses inflight.
From the mocha repository: mochajs/mocha#5148

If they update it, we will as well. For now, it seems to be out of my hands.

However, I did move chai to the developer dependencies, since it is only used for testing.
bed3ae0

@Lauren1066
Copy link
Collaborator

I've also just pushed an update to the docs on the setup page. It now tells people to use --production when using npm to install. This means the developer dependencies should not install. So, there should be no issues for regular users any longer.

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

No branches or pull requests

2 participants