IMHO it'd be useful to indicate the number of security and other updates available, rather than only listing them out.

CommentFileSizeAuthor
#1 drush-n1538164.patch1.64 KBdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new drush-n1538164.patch1.64 KB

Food for thought.

moshe weitzman’s picture

moshe weitzman commented
Assigned: Unassigned » jonhattan
Status: Needs review » Reviewed & tested by the community

Looks useful and correct to me.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

My only question is the *location* of the output appropriate, or should it be placed closer to the list (which would mean it goes in another function)?

jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented
Status: Reviewed & tested by the community » Needs work

It doesn't take into account updates to drupal core. I don't know if this is intended on your part but I think I prefer to know the total of updates.

Info on all updates is available $updateable, in the main function. It's cleaner to compute the total early in this stage to play better with other options as --simulate or --security-only.

I've no strong preference for the location to print the message but it should be printed for all of the cases.

greg.1.anderson’s picture

greg.1.anderson commented
Version: » 8.x-6.x-dev
Status: Needs work » Closed (won't fix)
Issue tags: +Needs migration

This issue was marked closed (won't fix) because Drush has moved to Github.

If this feature is still desired, you may copy it to our Github project. For best results, create a Pull Request that has been updated for the master branch. Post a link here to the PR, and please also change the status of this issue to closed (duplicate).

Please ask support questions on Drupal Answers.