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

Upgrade rimraf dependency to satisfy npm install warnings #44

Closed
zachleat opened this issue Jun 18, 2024 · 1 comment
Closed

Upgrade rimraf dependency to satisfy npm install warnings #44

zachleat opened this issue Jun 18, 2024 · 1 comment

Comments

@zachleat
Copy link

Looks like folks are getting npm install warnings from this package, is it possible to upgrade the rimraf package version? Or would you accept a PR of an upgrade? Looks like it might be a major version bump (the latest rimraf is v5).

Changelog here: https://github.com/isaacs/rimraf?tab=readme-ov-file#major-changes

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 warn deprecated [email protected]: Rimraf versions prior to v4 are no longer supported
npm warn deprecated [email protected]: Glob versions prior to v9 are no longer supported

Related: 11ty/eleventy#3299

@zachleat
Copy link
Author

We resolved this in Eleventy by forking: https://github.com/11ty/recursive-copy/

If you’d like these dependency updates merged back upstream, I’m happy to PR this but I think long term we’re trying to move towards Node-native fs.cp so I don’t blame y’all if this package is retired (11ty/eleventy#3360)

Thanks!

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