Review: Needs Fixing

LBTM. This is invalid XML, you have multiple root tags. The <content_attribute> 
has to be a child of <component>.

You can use the appstream package of your distribution to validate appstream 
files. E.g:

appstreamcli validate debian/widelands.appdata.xml

It throws a lot of errors, even before your change but you can still use it to 
see if new validation errors where introduced.
-- 
https://code.launchpad.net/~widelands-dev/widelands/oars_appdata/+merge/309624
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/oars_appdata.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to