ci(flatpak): add flatpak builder lint

[ReenigneArcher]

Description

Add flatpak manifest linting, per discussion from https://github.com/flathub/dev.lizardbyte.app.Sunshine/issues/36#issuecomment-2203990470

Screenshot

Issues Fixed or Closed

Type of Change

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] Dependency update (updates to dependencies)
• [x] Documentation update (changes to documentation)
• [x] Repository update (changes to repository files, e.g. .github/...)

Checklist

• [x] My code follows the style guidelines of this project
• [x] I have performed a self-review of my own code
• [x] I have commented my code, particularly in hard-to-understand areas
• [ ] I have added or updated the in code docstring/documentation-blocks for new or existing methods/components

Branch Updates

LizardByte requires that branches be up-to-date before merging. This means that after any PR is merged, this branch must be updated before it can be merged. You must also Allow edits from maintainers.

• [ ] I want maintainers to keep my branch updated

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 8.95%. Comparing base (acccacf) to head (9b4d191). Report is 136 commits behind head on master.

Additional details and impacted files

@@          Coverage Diff           @@
##           master   #2799   +/-   ##
======================================
  Coverage    8.95%   8.95%           
======================================
  Files          95      95           
  Lines       17386   17386           
  Branches     8268    8268           
======================================
  Hits         1557    1557           
  Misses      12964   12964           
  Partials     2865    2865           

Flag	Coverage Δ
Linux	6.78% <ø> (ø)
Windows	4.16% <ø> (ø)
macOS-12	10.01% <ø> (ø)
macOS-13	9.92% <ø> (-0.02%)	:arrow_down:
macOS-14	10.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[ReenigneArcher]

@gschintgen would you be able to test this PR? Mostly I'd like to ensure that deb or rpm packages didn't break due to any of the CMake changes.

[gschintgen]

Ok, sure, I'll install the .deb later (next 24h) and let you know.

On July 7, 2024 12:36:43 AM GMT+02:00, ReenigneArcher @.***> wrote:

@gschintgen would you be able to test this PR? Mostly I'd like to ensure that deb or rpm packages didn't break due to any of the CMake changes.

-- Reply to this email directly or view it on GitHub: https://github.com/LizardByte/Sunshine/pull/2799#issuecomment-2212032371 You are receiving this because you were mentioned.

Message ID: @.***>

[gschintgen]

I downloaded the last available deb from the CI here, that is the one before your (minor) last force push. It seems to work fine. I also checked out the PR locally and built it. I can't say there's anything out of the ordinary. But I only did very cursory testing (a few seconds of remote desktop.)

I had a quick look at sunshine.desktop and sunshine.service (since the respective .in files were modified) and they seem fine.