Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Remove dependency on rimraf #517

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

Merged
orta merged 1 commit into ds300:master from stianjensen:no-rimraf
Dec 14, 2024
Merged

Remove dependency on rimraf #517

orta merged 1 commit into ds300:master from stianjensen:no-rimraf
Dec 14, 2024

Conversation

Copy link
Contributor

@stianjensen stianjensen commented Mar 4, 2024

fs-extra was basically created to avoid having to install both mkdirp, rimraf etc, and is also already installed here, so we can avoid using both at the same time.

This moves rimraf to devDependencies, since it's still conveniant for the local 'clean' and 'prepack' commands.

At the same time, update to fs-extra 10.x, since it's lowest supported node version there is now still lower than the lowest supported version for patch-package, but has still introduced the option of using node's native fs.rm when available:
https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md

(fs-extra 11.x only works on node 14.14+, while patch-package currently supports all 14.x versions)

ZuBB, p2edwards, Gallmond, itotallyrock, erickzhao, Crystal-RainSlide, sairus2k, BenEinwechter, craiganderson-iotv, Rishi556, and 6 more reacted with thumbs up emoji
fs-extra was basically created to avoid having to install both mkdirp,
rimraf etc, and is also already installed here, so we can avoid using
both at the same time.
This moves rimraf to devDependencies, since it's still conveniant for
the local 'clean' and 'prepack' commands.
At the same time, update to fs-extra 10.x, since it's lowest supported
node version there is now still lower than the lowest supported version
for patch-package, but has still introduced the option of using node's
native fs.rm when available:
https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md
(fs-extra 11.x only works on node 14.14+, while patch-package currently
supports all 14.x versions)
Copy link

ZuBB commented Jun 5, 2024

just wanna add that rimraf throws a warn

npm WARN deprecated rimraf@2.7.1: Rimraf versions prior to v4 are no longer supported

Copy link
Contributor

Is this project still supported?

Copy link
Collaborator

orta commented Dec 14, 2024

Looks like fs-extra's removeSync is recursive, so this should be the same 👍🏻

@orta orta merged commit 40c0035 into ds300:master Dec 14, 2024
@stianjensen stianjensen deleted the no-rimraf branch December 15, 2024 00:41
Copy link

rakleed commented Jan 12, 2025

@orta could you please publish a new version with these changes?

sabinevidal, jennifer-shehane, dpeukert, theodorejb, icecream17, and niksy reacted with thumbs up emoji

Copy link

niksy commented Jun 12, 2025

Are there any blockers for getting this released? The only one I could think of is this:

fs-extra 11.x only works on node 14.14+, while patch-package currently supports all 14.x versions

/cc @orta

Copy link
Contributor Author

fs-extra 11.x only works on node 14.14+

This part is fine, since I only updated it to 10.x for now due to that.

niksy reacted with thumbs up emoji

Copy link

Kamahl19 commented Jul 8, 2025

@orta could you please make a release? Thanks!

techfg reacted with thumbs up emoji

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

Reviewers

1 more reviewer

@GSS-Rishi GSS-Rishi GSS-Rishi approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /