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

writeFile encoding has no effect any more #81

Open
fyears opened this issue Oct 17, 2021 · 1 comment
Open

writeFile encoding has no effect any more #81

fyears opened this issue Oct 17, 2021 · 1 comment

Comments

@fyears
Copy link
Contributor

fyears commented Oct 17, 2021

.putFileContents(filename, data /*{ format: encoding }*/)

Hi,

I forgot to mention an issue while I was refactoring webdav-fs to typescript.

I notice that writeFile, using putFileContents() from webdav-client, used to have encoding parameter.

However, the encoding option was removed from putFileContents().

Thus, I just kept the encoding parameter there to avoid changing end-user code. But actually it just "pass-through". And I do not have deep understanding about whether it's ok or not.

Is that an "expected change", or something that needs to be resolved?

@perry-mitchell
Copy link
Owner

Well, now that I look at it, I'm not sure if there's any change needed. webdav-fs is meant to somewhat mimic the standard fs interface of Node, and Node requires an encoding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants