consider moving cli to different lib

Question

consider moving cli to different lib

jonschlinkert opened this issue 10 years ago · 11 comments

I love using this, but I just realized that it's using meow, and I'm guessing that a good percentage of users like myself probably aren't using the CLI, but the dependency tree for it is absurdly massive. Especially considering this would have zero deps without it.

meow has 48 dependencies in total, this is its flattened dependency tree (along with the number of times each module appears in the tree):

{
  'trim-newlines': 1,
  'redent': 1,
  'strip-indent': 1,
  'get-stdin': 1,
  'indent-string': 1,
  'repeating': 1,
  'is-finite': 1,
  'number-is-nan': 1,
  'read-pkg-up': 1,
  'read-pkg': 1,
  'path-type': 1,
  'pinkie-promise': 4,
  'pinkie': 4,
  'pify': 2,
  'graceful-fs': 2,
  'normalize-package-data': 1,
  'load-json-file': 1,
  'strip-bom': 1,
  'is-utf8': 1,
  'parse-json': 1,
  'error-ex': 1,
  'find-up': 1,
  'path-exists': 1,
  'object-assign': 1,
  'validate-npm-package-license': 1,
  'spdx-expression-parse': 1,
  'spdx-license-ids': 2,
  'spdx-exceptions': 1,
  'spdx-correct': 1,
  'semver': 1,
  'is-builtin-module': 1,
  'builtin-modules': 1,
  'hosted-git-info': 1,
  'minimist': 1,
  'loud-rejection': 1,
  'camelcase-keys': 1,
  'map-obj': 1,
  'camelcase': 1
}

Answer 1 · 2015-10-31T15:45:34.000Z

ping, I'd be happy to do a pr to remove this dependency. Unfortunately this shows up every time we install a gulp plugin that uses gulp-util, so it's creating a ton of extra files that aren't needed.

Answer 2 · 2015-11-05T15:49:29.000Z

Great idea, also getting a lot of extra files when installing via npm. Perhaps a separate dateformat-cli package depending on this?

Answer 3 · 2015-11-06T09:25:50.000Z

I have now published a new npm package "dateformat-light" that removes the CLI, and fixes the AMD inclusion bug

Answer 4 · 2015-11-06T13:05:35.000Z

well, I'm a fan of small modules, but that doesn't really solve this issue though, since there are many gulp plugins using gulp-util, and gulp-util uses this lib. The issue will need to be resolved here unfortunately

Answer 5 · 2016-01-25T03:46:29.000Z

fwiw, since this library is no longer supported, I'll be using dateformat-light going forward.

Answer 6 · 2016-06-11T16:07:08.000Z

@felixge can I please get you to consider removing meow from deps?

Answer 7 · 2016-06-19T12:16:14.000Z

@jonschlinkert send a PR and I'll merge it. Ping me in the PR, I don't watch this repo.

Answer 8 · 2016-06-19T14:24:22.000Z

Great thanks!

Answer 9 · 2016-06-19T18:27:35.000Z

Thanks for working on this @jonschlinkert! Please keep us posted.

Answer 10 · 2016-06-20T04:56:42.000Z

no prob, I'll be sending a pr tomorrow (Monday)

Answer 11 · 2020-12-04T08:26:35.000Z

Closing as meow is now removed as a dependency