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
add missing fs.readdir encoding option #84
Conversation
fs.readdir has optional option called encoding: https://nodejs.org/api/fs.html#fs_fs_readdir_path_options_callback
Fails tests. I thikn it's supposed to accept an object with an 'encoding' property, not an encoding string. |
missing encoding argument for readdir function
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.
there was a missing parameter for the function readdir
readdir accept object or string but the issue in the previous patch was because of a missing argument, sorry it was an undo before commit by mistake |
Adding the param arbitrarily breaks older versions that did not take this parameter. Different approach in #93 that is agnostic about the existence of the parameter and its type. Can you check to see if that meets your needs? |
ok i understand btw thanks for this module :) |
i think you can close this request |
Make `fs.Stats` `.uid` and `.gid` fields always be unsigned. Pass through optional arguments to `fs.readdir`. Previously they were dropped. Fixes: #13918 Fixes: isaacs/node-graceful-fs#84
fs.readdir has optional option called encoding:
https://nodejs.org/api/fs.html#fs_fs_readdir_path_options_callback