-
Notifications
You must be signed in to change notification settings - Fork 152
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
fs.unlink should probably be fs.unlinkSync #171
Comments
Odd, I don't know why nobody has run into that before, and the test suite passes without it. Maybe something with Node versions? At any rate I applied your change. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Was getting the error below:
Per the error, the problem was with line 211, which runs
fs.unlink(screenshot)
. Looks likefs.unlink
expects a second param (a callback), so fixed it by changing tofs.unlinkSync(screenshot)
.The text was updated successfully, but these errors were encountered: