emberjs / rfcs

RFCs for changes to Ember
https://rfcs.emberjs.com/
791 stars 406 forks source link

Remove ember-cli-dependency-checker #966

Open NullVoxPopuli opened 1 year ago

NullVoxPopuli commented 1 year ago

fewer dependencies = better?

wagenet commented 1 year ago

I've found this package helpful in two scenarios recently: 1) I forgot to run pnpm install 2) I set an override in my package.json and forgot to update the actual specified dependency version to match.

NullVoxPopuli commented 1 year ago

I've added those scenarios to the RFC.

  1. is mostly covered, I think? you get a pretty early error otherwise
    1. there is another use case here where your dependencies are stale.
  2. this is uncovered

maybe instead of removing this dependency across the board, we

SergeAstapov commented 1 year ago

+1 to callout from @wagenet as not once it helped to avoid mistakes by not running npm i/pnpm i