dooboolab-community/dooboo-cli

Removing path and fs as dependencies.

ZeroCho opened this issue · 4 comments

ZeroCho commented

Is there any special reason for using npm's path and fs modules rather than default ones node provides?

path
fs
can be removed, so that dooboo-cli use native modules node provides.

hyochan commented

Again good point. I was willing to ensure that they are installed correctly as discussed in issue. However, if you are confident that those are not needed we can get that away!

ZeroCho commented

Moreover, path & fs modules currently installed are out of date(almost 3~4 yrs). Way better use default modules node provides

  • 👍1
  • 😄1
hyochan commented

Just released 1.2.1 with this update. Let's see how it goes.
Thanks for the PR. The first one to contribute to dooboo-cli!