quaertym / ember-cli-dependency-checker

Ember CLI addon for checking missing node and bower dependencies before running ember commands
MIT License
40 stars 38 forks source link

Update Dependency Error Message? #8

Closed brzpegasus closed 10 years ago

brzpegasus commented 10 years ago

The addon currently displays "Missing npm packages" and "Run npm install to install missing dependencies" (same with Bower). However, the error might be a bit misleading because sometimes, the packages are installed; they're just not the right version.

Also, I realize this is a bit subjective, but it'd be nice to minimize the colors used in the error message (replace green with white?):

screen shot 2014-10-31 at 5 30 33 pm

quaertym commented 10 years ago

npm install is used for the update as well, right? Which word is misleading do you think, missing or install ?

And this is how it looks like on my screen, it should keep the default color I guess for Package: and versions. screen shot 2014-11-01 at 2 07 17 am

brzpegasus commented 10 years ago

Ah, yeah, default color for those might be best.

As for the text, I was referring to the word "missing". Maybe "unmet" would be a better choice? It's not a big deal, just a bit of nit pickiness on my part.