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

Remove/replace dependency on MV #719

Open
nielm opened this issue May 23, 2024 · 3 comments
Open

Remove/replace dependency on MV #719

nielm opened this issue May 23, 2024 · 3 comments

Comments

@nielm
Copy link

nielm commented May 23, 2024

npm package MV has not been updated in 9 years, and depends on a deprecated package inflight which leaks memory:

Bunyan should find a replacement for mv.

> npm install --save
npm warn deprecated [email protected]: 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.
> npm ls inflight
my-package
└─┬ [email protected]
  └─┬ [email protected]
    └─┬ [email protected]
      └─┬ [email protected]
        └── [email protected]
@nielm
Copy link
Author

nielm commented May 23, 2024

Workaround: add the following to package.json:

  "overrides": {
    "mv": "npm:dry-uninstall"
  },

which will prevent mv from being installed by using the dummy package dry-uninstall instead.

Mv is an optional module used for "rotating-file" stream type

@TWiStErRob
Copy link

Interesting workaround! Thanks for sharing.

I also found the same problem: renovatebot/renovate#29265, any chance you're coming from Renovate too?

@aabmass
Copy link

aabmass commented Jun 3, 2024

Related/duplicate #667

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

3 participants