search

populist / drupal-auto-update

A little bit of magic for automatically updating Drupal sites
27 stars 14 forks source link

Unreliable pm-updatestatus result #3

Open shauntyndall opened 6 years ago

shauntyndall commented 6 years ago

I'm finding the following line of code is returning a false positive result when module names are long. See the screenshot. Would another result format work better? of possibly using the --pipe option instead?

PLUGIN_UPDATES="$(terminus drush $SITE_UUID.$TERMINUS_ENV -- pm-updatestatus --format=list | grep -v ok)"

image

BTW, thanks for this. There is real value in the concept.

shauntyndall commented 6 years ago

PLUGIN_UPDATES="$(terminus drush $SITE_UUID.$TERMINUS_ENV -- pm-updatestatus --pipe)"

Worked better for my scenario. Not confident enough in the solution to make a PR.