Skip to content

Conversation

@anomiex
Copy link

@anomiex anomiex commented Mar 26, 2025

Resolves deprecation warnings about glob, inflight, and rimraf. See istanbuljs/nyc#1563.

Resolves deprecation warnings about glob, inflight, and rimraf.
See istanbuljs/nyc#1563.
Copy link
Member

@bcoe bcoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This version of rimraf requires Node 14 or higher I believe? So this is a breaking change. Not necessarily a problem, but we will need to be careful about landing.

Update the PR title to:

fix(deps)!: upgrade rimraf addressing upstream warnings

If you don't mind 👏

@anomiex anomiex changed the title chore(deps): Bump rimraf to v4 fix(deps)!: upgrade rimraf addressing upstream warnings May 2, 2025
@anomiex
Copy link
Author

anomiex commented May 2, 2025

If you don't mind 👏

I dont't mind at all. 🙂

"istanbul-lib-coverage": "^3.2.0",
"p-map": "^3.0.0",
"rimraf": "^3.0.0",
"rimraf": "^4.0.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the slow turnaround on this review, if you don't mind also add this to package.json:

  "engines": {
    "node": ">=14"
  },

I can take it from there.

@anomiex
Copy link
Author

anomiex commented May 7, 2025

Done, thanks!

anomiex added a commit to anomiex/spawn-wrap that referenced this pull request May 7, 2025
anomiex added a commit to anomiex/spawn-wrap that referenced this pull request May 7, 2025
@mertssmnoglu
Copy link

Hi, is there any progress on this PR? I'm receiving 8 lines of deprecation warnings in each clean npm install command in my projects.

@anomiex
Copy link
Author

anomiex commented Jul 9, 2025

@bcoe Is there anything else you need from me to move forward on this?

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

Successfully merging this pull request may close these issues.

3 participants