Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Dec 31, 2023

  • Add the <developer><name> tag
  • Mark the <developer_name> tag as deprecated
  • Remove the second appstreamcli validation
  • Improve appstreamcli arguments

- Add the `<developer><name>` tag
- Mark the `<developer_name>` tag as deprecated
- Remove the second appstreamcli validation
- Improve appstreamcli arguments
@ghost ghost requested a review from sonnyp as a code owner December 31, 2023 20:10
@Hofer-Julian Hofer-Julian merged commit 8be2340 into workbenchdev:main Jan 7, 2024
@Hofer-Julian
Copy link
Contributor

Thanks

<name translatable="no">Workbench</name>
<!-- developer_name tag deprecated with Appstream 1.0 -->
<developer_name translatable="no">Sonny Piers</developer_name>
<developer id="gitlab.com">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is incorrect - github.com is not the developer of Workbench

The element should have a id property, containing a unique ID to identify the respective developer. It is recommended to use a web URL, like gnome.org as ID to achieve a higher chance of uniqueness.

https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-developer

I will fix

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure how to use the new tag.

ximion/appstream#575

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants