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
Remove fs-extra
dependency
#597
Conversation
a84c31b
to
9f639d4
Compare
Tests are failing |
50692f5
to
9f639d4
Compare
Umm, cannot reproduce in my local environment... 👀 |
9f639d4
to
10b6780
Compare
10b6780
to
eb62414
Compare
Thanks :) |
This reverts commit 098a594.
Can you guys elaborate on the benefit of reducing dependencies? I guess install is slightly faster.. and yeah "fewer dependencies" makes it more robust I wish node returned file contents as utf8 strings by default... maybe we should have a I'm working on some scripts of my own and just curious what you guys think is "good" or "best" practice here |
The usage of
fs-extra
is only related to reading/writing JSON currently, then they can be replaced with some combinations of native modules to reduce the dependency.