Skip to content

feat: add AppStream stock icons for multiflexi and multiflexi-common#106

Merged
Vitexus merged 1 commit into
mainfrom
feat/appstream-stock-icons
May 18, 2026
Merged

feat: add AppStream stock icons for multiflexi and multiflexi-common#106
Vitexus merged 1 commit into
mainfrom
feat/appstream-stock-icons

Conversation

@Vitexus

@Vitexus Vitexus commented May 18, 2026

Copy link
Copy Markdown
Member

Summary

  • Add debian/cz.vitexsoftware.multiflexi.metainfo.xml with <icon type="stock">multiflexi</icon> for the multiflexi package
  • Add debian/multiflexi.install to install multiflexi.svg into usr/share/icons/hicolor/scalable/apps/ and the metainfo into usr/share/metainfo/
  • multiflexi-common was already correctly configured; no changes needed there

Test plan

  • Build both packages and verify multiflexi.svg lands in /usr/share/icons/hicolor/scalable/apps/multiflexi.svg
  • Verify /usr/share/metainfo/cz.vitexsoftware.multiflexi.metainfo.xml is installed in the multiflexi package
  • Run appstreamcli validate /usr/share/metainfo/cz.vitexsoftware.multiflexi.metainfo.xml — expect no errors
  • Confirm multiflexi-common still installs its icon and metainfo correctly

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Added AppStream metadata to enable proper application discovery in system package managers and application launchers.
    • Configured installation of application icon to system directories for consistent desktop environment integration.

Review Change Stack

…packages

Install multiflexi.svg and multiflexi-common.svg as AppStream stock icons
for their respective packages via hicolor icon theme and metainfo XML.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 18, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 054bf09d-0750-4385-a4ac-fc5032b1c471

📥 Commits

Reviewing files that changed from the base of the PR and between 4d3d795 and 1ec5e12.

📒 Files selected for processing (2)
  • debian/cz.vitexsoftware.multiflexi.metainfo.xml
  • debian/multiflexi.install

📝 Walkthrough

Walkthrough

This PR adds AppStream metadata for the MultiFlexi application by introducing a component metainfo XML file and updating the Debian packaging to install both the metadata and application icon to their respective system directories.

Changes

AppStream metadata and packaging integration

Layer / File(s) Summary
AppStream metainfo definition and installation mapping
debian/cz.vitexsoftware.multiflexi.metainfo.xml, debian/multiflexi.install
AppStream metainfo XML defines the cz.vitexsoftware.multiflexi component with MIT licensing, application name/summary/description, homepage and bugtracker URLs, VitexSoftware developer identity, content rating, and icon reference. Debian install mappings configure the metainfo file and multiflexi.svg icon for installation to /usr/share/metainfo/ and hicolor scalable icons directory respectively.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A metadata bloom in springtime!
Icons and info in perfect time,
AppStream declares the MultiFlexi way,
Debian packages now brighten the day,
System integration, smooth and sublime!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/appstream-stock-icons

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Vitexus Vitexus merged commit 1c7f22f into main May 18, 2026
1 of 5 checks passed
@Vitexus Vitexus deleted the feat/appstream-stock-icons branch May 18, 2026 19:48
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.

1 participant