-
Notifications
You must be signed in to change notification settings - Fork 17
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
Metadata: parse new names #181
Conversation
- PackageName || Title - PackagerName || PackagedBy - Icon || PackageIcon - Url || Links
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.
LGTM
Thank you very much, will overhaul the section "field descriptions" of Metadata.md in the upcoming days.
Thank you very much, LGTM |
@nephros, unfortunately this change fails to compile: Can you please take a look at it. |
Damn, missing closing bracket. Sorry about that. Fixes coming up! |
… which I failed to see (despite being trivial) due to being overly tired when I detected that building fails.
Never mind. It uncovered another issues: The CI runs at GitHub show success, even though compiling fails! I have to improve the workflow definition(s).
|
References: - [Issue \#100 "Draft a version of the metadata specification"](#100) - [PR \#181 "Metadata: parse new names" for the SailfishOS:Chum GUI application](sailfishos-chum/sailfishos-chum-gui#181).
…ion (#109) * Replace four tags and document this in the text References: - [Issue \#100 "Draft a version of the metadata specification"](#100) - [PR \#181 "Metadata: parse new names" for the SailfishOS:Chum GUI application](sailfishos-chum/sailfishos-chum-gui#181). * Update link to final metadata definition v0 * Add proper headers
Implements parsing new tags
See sailfishos-chum/main#100
As the implementor I have taken the liberty of supporting PackagedBy as well, because I like it.