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

Prevent callback from being called twice #145

Closed
wants to merge 1 commit into from
Closed

Conversation

RyanZim
Copy link

@RyanZim RyanZim commented May 4, 2017

I'm one of the maintainers of fs-extra. At fs-extra we maintain a fork of rimraf internally. A user reported a bug in rimraf; I applied a patch to our fork, and I'm forwarding the patch upstream.

Here is the issue discussion: jprichardson/node-fs-extra#392

@RyanZim
Copy link
Author

RyanZim commented Jun 10, 2017

@isaacs ping?

@chrisblossom
Copy link

@isaacs Is there a reason this PR hasn't been merged?

@RyanZim
Copy link
Author

RyanZim commented Jun 3, 2020

@isaacs bump

@benmbpi
Copy link

benmbpi commented Jul 13, 2022

@isaacs we're noticing same issue, any update on when this can be merged?

@isaacs
Copy link
Owner

isaacs commented Jan 13, 2023

No longer applicable in v4.

@isaacs isaacs closed this Jan 13, 2023
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.

4 participants