-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better AppStream metadata #14648
base: master
Are you sure you want to change the base?
Better AppStream metadata #14648
Conversation
|
||
<translation type="gettext">minetest</translation> | ||
|
||
<update_contact>sfan5@live.de</update_contact> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use this opportunity to put someone else's email here.
Volunteers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A Minetest mailing list (for core devs?) would be convenient for such cases.
I have added a value to display_length. The goal is to make it appear as supported for both PC and mobile devices, so I picked 360 which is the value used by GNOME Calculator. I have also added a tag to display the offline use of Minetest possible. |
I consider this PR as finished, IMO the last TODO item isn't really important. The last thing to check is the contact email, but don't know what to put here obviously. |
ruben and c55 are also listed here: https://github.com/minetest/minetest/security/policy |
Fix #14647
To do
This PR is Ready for Review
.appdata.xml
to.metainfo.xml
(.metainfo.xml is the recommended file name).desktop
.desktop
descriptiongamepadand touch.desktop
file, remove.desktop
from the top id (no idea why it was like that, didn't see it anywhere in the spec)bugtracker
anddonation
How to test
Try running:
or this (less strict):
Warning
With the renaming of the AppStream file, the Flathub manifest will need to be updated to reflect the change I think