yeoman / update-notifier

Update notifications for your CLI app
BSD 2-Clause "Simplified" License
1.76k stars 132 forks source link

Package deprecation notice #56

Open ghost opened 9 years ago

ghost commented 9 years ago

Could the update-notifier package also post a console update notification when a CLI tool has been deprecated using npm's deprecate command?

arthurvr commented 9 years ago

IMHO this is out of scope for this module.

ghost commented 9 years ago

Is it out of scope to print the deprecation notice for a CLI tool that is simply out of date, when the out of date version has been deprecated. Especially if the deprecation notice contains text like npm's example: critical bug fixed in v0.2.3

This seems like critical information we should be disseminating to consumers of these CLI tools.

sindresorhus commented 9 years ago

I think it's a great idea. PR welcome :)

robbbz commented 9 years ago

Can we make this a separate module? I can see apps where I wouldn't want update notifications but I do want notifications when it's deprecated.

ghost commented 9 years ago

@sindresorhus based on @robbbz comment, what do you think?

I am still in favor of update-notifier providing deprecation notices under the run npm text. I'm also fine with having a separate package that will just retrieve the deprecation notice of a particular package at a particular version (kind of like the helper package latest-version).

However, should we also have a deprecation-notifier package that provides similar functionality to update-notifier?