Differential D14034

Add caption to screenshot in AppStream data
ClosedPublic
Actions

Authored by rkflx on Jul 10 2018, 8:39 PM.

Details

Reviewers
ngraham
Group Reviewers
Spectacle
Commits
R166:4ae2b7aec28a: Add caption to screenshot in AppStream data
Summary

…and bring Appstreamercli warnings on the CI down to zero.

Test Plan

appstreamcli validate --pedantic desktop/org.kde.spectacle.appdata.xml

Diff Detail

Repository
R166 Spectacle
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
rkflx requested review of this revision.Jul 10 2018, 8:39 PM
rkflx created this revision.
rkflx added a comment.Jul 11 2018, 6:42 PM

@ngraham Thanks for accepting the Gwenview patch. For Spectacle I assume you are still pondering over a better wording?

ngraham accepted this revision.Jul 12 2018, 1:08 PM

Oops, the email notifications for these two patches were so similar that I assumed they were the more common case of duplicate emails rather than separate patches! This one looks just fine too.

This revision is now accepted and ready to land.Jul 12 2018, 1:08 PM
Closed by commit R166:4ae2b7aec28a: Add caption to screenshot in AppStream data (authored by rkflx). · Explain WhyJul 12 2018, 10:58 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 37672

View Options

desktop/org.kde.spectacle.appdata.xml

Loading...