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

fix fs.read bug since node.js is upgrade #251

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kundaiAtPropertyMe
Copy link

node.js 8 remove the old fs.read format which cause the function failed with error message Fatal error: Second argument needs to be a buffer

fix the write function for it.

node.js 8 remove the old fs.read format which cause the function failed with error message Fatal error: Second argument needs to be a buffer

fix the write function for it.
@ibirman
Copy link

ibirman commented Jan 16, 2018

👍

@psprogis
Copy link

hi guys, could you please merge these changes and publish new version on npm ?

LittleWhiteYA added a commit to LittleWhiteYA/crawler that referenced this pull request May 26, 2018
xpolakm5 added a commit to xpolakm5/restler that referenced this pull request May 28, 2018
@skolesnyk
Copy link

Common, what takes so much time to push a new this commit to master?

@jc21
Copy link

jc21 commented Oct 10, 2019

Still getting this problem and it's almost 2020 ...

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.

5 participants