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 check for fs.realpath.native support, since it's everywhere #887

Merged
merged 1 commit into from
Apr 6, 2021

Conversation

RyanZim
Copy link
Collaborator

@RyanZim RyanZim commented Apr 3, 2021

No description provided.

@RyanZim RyanZim requested review from JPeer264 and manidlou April 3, 2021 19:54
@RyanZim RyanZim merged commit f4a880d into master Apr 6, 2021
@RyanZim RyanZim deleted the ryan/realpath-native branch April 6, 2021 13:02
geralfonso pushed a commit to geralfonso/node-fs-extra that referenced this pull request Jan 5, 2022
RyanZim added a commit that referenced this pull request Apr 16, 2022
* Revert "Remove check for fs.realpath.native support, since it's everywhere (#887)"

This reverts commit f4a880d.

* Added warning and fallback support for missing fs.realpath.native

Signed-off-by: Lam Wei Li <peteriman@mail.com>

* Removed NodeJS check and fs.realpath.native fallback

Signed-off-by: Lam Wei Li <peteriman@mail.com>

* Leaving original listener intact during test and better phasing for warning

Signed-off-by: Lam Wei Li <peteriman@mail.com>

* Using done() for proper async test

Signed-off-by: Lam Wei Li <peteriman@mail.com>

* Updated comments for clarity

Co-authored-by: Ryan Zimmerman <17342435+RyanZim@users.noreply.github.com>

Co-authored-by: Ryan Zimmerman <17342435+RyanZim@users.noreply.github.com>
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.

2 participants