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

Metadata: parse new names #181

Merged
merged 2 commits into from
Apr 2, 2023
Merged

Metadata: parse new names #181

merged 2 commits into from
Apr 2, 2023

Conversation

nephros
Copy link
Contributor

@nephros nephros commented Mar 23, 2023

Implements parsing new tags

  • PackageName || Title
  • PackagerName || PackagedBy
  • Icon || PackageIcon
  • Url || Links

See sailfishos-chum/main#100

As the implementor I have taken the liberty of supporting PackagedBy as well, because I like it.

nephros added 2 commits March 23, 2023 07:54
 - PackageName  || Title
 - PackagerName || PackagedBy
 - Icon         || PackageIcon
 - Url          || Links
Copy link
Collaborator

@Olf0 Olf0 left a 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.

@rinigus
Copy link
Contributor

rinigus commented Mar 25, 2023

Thank you very much, LGTM

@nephros nephros marked this pull request as ready for review March 25, 2023 08:07
@Olf0 Olf0 merged commit 2de3d79 into sailfishos-chum:main Apr 2, 2023
@nephros nephros deleted the new-meta branch April 2, 2023 15:32
@Olf0
Copy link
Collaborator

Olf0 commented Apr 2, 2023

@nephros, unfortunately this change fails to compile:
chum-gui-0.6.0_compiling-fails.txt

Can you please take a look at it.

@nephros
Copy link
Contributor Author

nephros commented Apr 3, 2023

Damn, missing closing bracket. Sorry about that.

Fixes coming up!

@nephros nephros restored the new-meta branch April 3, 2023 07:10
@nephros nephros deleted the new-meta branch April 3, 2023 07:10
@nephros nephros restored the new-meta branch April 3, 2023 07:11
@nephros nephros mentioned this pull request Apr 3, 2023
Olf0 pushed a commit that referenced this pull request Apr 3, 2023
fixup! parse new names:

Co-authored-by: @nephros (nephros <nemo@pgxperiiia10>)
@Olf0
Copy link
Collaborator

Olf0 commented Apr 3, 2023

Damn, missing closing bracket.

… which I failed to see (despite being trivial) due to being overly tired when I detected that building fails.

Sorry about that.

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).

Fixes coming up!

Thank you for PR #185. Merged and compiling, as expected.

Olf0 added a commit to sailfishos-chum/main that referenced this pull request Apr 4, 2023
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).
Olf0 added a commit to sailfishos-chum/main that referenced this pull request Apr 6, 2023
…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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants