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
unix,win: add uv_fs_statfs() #2396
Conversation
Looking good so far! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UNIX side LGTM!
LGTM |
2c2df83
to
4f9bd65
Compare
Added the rest of the missing pieces. PTAL. CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/1476/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Fixes: libuv#2386 PR-URL: libuv#2396 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Bartosz Sosnowski <bartosz@janeasystems.com>
CI after resolving conflicts and squashing: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/1478/ |
Fixes: #2386