Feat/flatpak#265
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@plind-junior Please review |
7df5895 to
99b5dcb
Compare
|
@plind-junior Please review |
99b5dcb to
f067a31
Compare
What changed
Adds
desktop/flatpak/with acom.vouchdev.vouchFlatpak manifest onorg.freedesktop.Platform//23.08, avouch-review-uilauncher,.desktopentry, AppStream metainfo, hicolor icons, Flathub submission template, build/validate scripts,tests/test_flatpak*.py, aflatpakCI workflow, anddocs/desktop-flatpak.md. Also adds a[desktop]optional extra (alias of[web]) and amake flatpaktarget.Closes #211
Why
Fedora/Arch users without snap need a sandboxed Linux install path for the review console (#211). Flatpak covers that audience with
--filesystem=homefor.vouch/trees under$HOMEand--share=networkfor the localhost review-ui server. Flathub submission assets are included so the app can be published ascom.vouchdev.vouchonce approved upstream.What might break
Nothing for existing
.vouch/directories or kb.* behaviour. This is additive packaging only — no on-disk layout, protocol, or runtime changes for pipx/CLI users.VEP
Not required — no object model, kb.* method, on-disk layout, bundle format, or audit-log shape changes.
Tests
make checkpasses locally (lint + mypy + pytest)tests/test_flatpak.py,tests/test_flatpak_validate_extended.py)CHANGELOG.mdupdated under## [Unreleased]