Code review comment for ~ahasenack/ubuntu/+source/update-notifier:groovy-update-notifier-esm-aware

Revision history for this message
Bryce Harrington (bryce) wrote :

LGTM.

Just a suggestion, but you may also want to mention in the changelog that UbuntuESMApps is being differentiated from UbuntuESM. Not sure how to phrase that in a non-confusing way, but that seems like a notable change. At first glance, I thought this was squashing two unrelated changes.

review: Approve

« Back to merge proposal