D28648: Fix AppStream information
Pino Toscano
noreply at phabricator.kde.org
Tue Apr 7 12:32:55 BST 2020
pino added a comment.
Can you please explicitly say what are the changes (and possibly why)? The reason is that saying "fix $file so that $tool gives no errors" is valid today, may be valid tomorrow, however there is the risk that will be useless in 1/2/5 years on. Tools come and go, and even the reporting they produce change over time.
IMHO something like:
appstream: fix screenshot URL and size
Use the canonical location for the screenshot, and fix width/height to match its actual size.
would be way better.
(BTW the current URL works fine for me.)
REPOSITORY
R204 KAddressBook
REVISION DETAIL
https://phabricator.kde.org/D28648
To: ognarb, #kde_pim
Cc: pino, kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20200407/257055af/attachment.html>
More information about the kde-pim
mailing list