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

#256: Provide handling for ctx in maybeCallback, fixes changes to fs.… #259

Closed
wants to merge 1 commit into from

Conversation

pirog
Copy link

@pirog pirog commented Jan 11, 2019

…existsSync introduced in later node 10 versions

@tschaub, my understanding of this level of node is pretty poor so no idea whether this is actually smart or not but currently using this as a workaround for #256 and wanted to throw it up here for your judgment/a starting point for others

… to fs.existsSync introduced in later node 10 versions
@tschaub
Copy link
Owner

tschaub commented Feb 4, 2019

Thanks for the effort on this @pirog. The 4.8.0 release includes an alternative implementation from @huochunpeng (see #260).

@tschaub tschaub closed this Feb 4, 2019
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