Skip to content
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

appdata: Update appdata #110

Merged
merged 1 commit into from Dec 27, 2023
Merged

appdata: Update appdata #110

merged 1 commit into from Dec 27, 2023

Conversation

ghost
Copy link

@ghost ghost commented Dec 27, 2023

  • Use appstreamcli to validate appdata
  • Fix some appdata issues
  • Mark the developer_name tag as untranslatable also deprecated
  • Remove useless custom Purism tags
  • Add vcs-browser and translate URLs
  • Add newly added developer > name tag
  • Remove duplicated translation tag

- Use appstreamcli to validate appdata
- Fix some appdata issues
- Mark the developer_name tag as untranslatable also deprecated
- Remove useless custom Purism tags
- Add vcs-browser and translate URLs
- Add newly added developer > name tag
- Remove duplicated translation tag
@vixalien
Copy link
Owner

Thanks as always Sabri! I really appreacite your work!

@vixalien vixalien merged commit f5216f0 into vixalien:main Dec 27, 2023
2 checks passed
@ghost ghost deleted the fix-appdata branch December 27, 2023 04:11
@ghost
Copy link
Author

ghost commented Dec 27, 2023

I tested using flatpak json file, adding the "run-tests" : true, string after the following lines.

before

"name": "sticky-notes",
"buildsystem": "meson",

after

"name": "sticky-notes",
"buildsystem": "meson",
"run-tests" : true,

result

[1/2] Running all tests.
1/3 Validate desktop file          OK              0.02s
2/3 Validate schema file           OK              0.01s
3/3 Validate appstream file        OK              0.02s

@vixalien
Copy link
Owner

Ah, I will need to add run-tests then..

@ghost
Copy link
Author

ghost commented Dec 27, 2023

Not exacly. It usefull for the devel json

@vixalien
Copy link
Owner

Yes, I meant I need to add it to the devel flatpak manifest

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.

None yet

1 participant