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
feat(fs): support FileInfo.dev on Windows #18073
Conversation
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.
lstating a dangling file seems to be failing on Windows. Presumably that's still related to these changes?
174b54a
to
07cde82
Compare
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!
Yay! Can we revert the part of this change that sets everything to 0 for windows? The null->zero should happen in ext/node. Also this is a breaking API change. |
Addresses feedback from #18073 (comment). Reverts changes to `FileInfo` fields that are not available on Windows making them `null`. Only `FileInfo.dev` is non-null.
This commit adds support for retrieving
dev
informationwhen stating files on Windows.
Additionally
Deno.FileInfo
interfaces was changed to alwaysreturn 0 for fields that we don't retrieve information for on Windows.
Closes #18053