search

reworkcss / rework-npm

Import CSS from npm modules using rework
BSD 2-Clause "Simplified" License
87 stars 19 forks source link

Implement a command line interface. #20

Closed Siilwyn closed 9 years ago

Siilwyn commented 9 years ago

By supporting a command line interface this package could be directly used with npm scripts. What do you think of the idea?

conradz commented 9 years ago

There already is one: https://github.com/sethvincent/rework-npm-cli. Often you will want to use other rework plugins along with this, which is very difficult with a CLI script.

Siilwyn commented 9 years ago

Yes I saw that one but I figured having CLI functionality on the original package is much better for the following reasons:

  1. Having less dependencies.
    In this case three less.
  2. Being up to date.
    Who knows what the future holds.
  3. Not having to rely on small packages.
    With only two maintainers the chances of an abandoned package are bigger.
  4. Not using 'poor' packages.
    The rework-npm-cli package depends on a deprecated package. Doubts about the quality.

Well I hope I convinced you, I don't see any negative consequences to implementing a CLI apart from not being used much because people combine it with other rework plugins? Let's say you still wouldn't find it worth your time to implement would you accept a pull request implementing it?