Skip to content

ci(release): release libra binary and monobean installer for windows_x64,linux_x86_64 and macos_aarch64#998

Merged
genedna merged 23 commits into
gitmono-dev:mainfrom
yyk808:main
May 1, 2025
Merged

ci(release): release libra binary and monobean installer for windows_x64,linux_x86_64 and macos_aarch64#998
genedna merged 23 commits into
gitmono-dev:mainfrom
yyk808:main

Conversation

@yyk808

@yyk808 yyk808 commented May 1, 2025

Copy link
Copy Markdown
Collaborator

This pull request introduces several updates across multiple areas, including platform-specific installation logic, release workflow enhancements, dependency and logging improvements, and user interface updates for the Monobean application. The most significant changes include making platform checks more flexible, refining the release workflows, and adding new features like logging to file and GTK4 version upgrades.

TLDR;

  • Release libra binary and monobean installer for windows_x64,linux_x86_64 and macos_aarch64.
  • Artifacts generated are put in a draft.
    {35D6EC3E-3347-4210-8D61-982248B6E156}
    {ED46E545-45EC-4367-B04A-3931123E6021}
    image
1f785a1586936fe64ffdbe59e4c6185

Known bugs

  • On windows and macos, monobean cannot found some icons.
  • On macos, monobean installer doesn't contain dylib itself, manually installing some dependencies are still needed FOR NOW.
  • On ubuntu-latest(also my wsl2 ubuntu), it shows an unknown error when packaging Arch Linux pacman package using makepkg. However, the same script works on Arch Linux host machine.
==> ERROR: An unknown error has occurred. Exiting...
  • Windows monobean installer will stick its version to 0.0.1, or we can't locate its package name in github workflows.
  • libra binary is SOOOOOOOOOO big on linux.

Platform-specific Installation Logic Updates:

  • Updated platform checks in .github/install-dep/action.yml to use the contains function for broader matching of platform names, replacing exact equality checks. [1] [2] [3] [4] [5]

Release Workflow Enhancements:

  • Adjusted .github/workflows/release.yml to handle semantic versioning tags (v*.*.*) and added architecture-specific packaging for Linux, macOS, and Windows using cargo-packager. Removed the create-release job and replaced binary upload steps with platform-specific preparation and release steps. [1] [2]
  • Introduced packaging configuration files (packager_x86_64-unknown-linux-gnu.toml, packager_aarch64-apple-darwin.toml, packager_x86_64-pc-windows-msvc.toml) for generating platform-specific installers. [1] [2] [3]

yyk808 and others added 20 commits April 22, 2025 15:06
This commit adds support for the --amend flag in the commit command, allowing users to amend the previous commit instead of creating a new one.

This feature enhances the CLI by enabling more flexible version history management, especially useful when users want to fix typos or update commit messages without generating extra commits.

and modify the docs

--amend only support when the commit has only a parent commit.
Signed-off-by: Quanyi Ma <eli@patch.sh>
Signed-off-by: Quanyi Ma <eli@patch.sh>
Signed-off-by: Eli Ma <eli@patch.sh>
Signed-off-by: Quanyi Ma <eli@patch.sh>
@vercel

vercel Bot commented May 1, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mega ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2025 3:35am

@genedna
genedna requested a review from Copilot May 1, 2025 02:47
@genedna genedna added the client label May 1, 2025
@genedna genedna added this to the 0.3.0 - Decentralized Git milestone May 1, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the release workflows, installer configurations, and application logging to support a multi-platform release of the Monobean application while enhancing dependency setup and packaging logic. Key changes include:

  • Refined platform checks and caching conditions in install and workflow scripts.
  • Improved logging setup in the application and updated resource loading logic.
  • Updated packaging configuration files for Linux, Windows, and macOS.

Reviewed Changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
monobean/src/main.rs Updates resource loading logic with a fallback path.
monobean/src/application.rs Modifies logging configuration with conditional setup for debug/release.
monobean/setup.py Integrates an upgrade flag via argparse and refines setup behaviors.
monobean/resources/*.toml Adds configuration files for platform-specific packaging.
monobean/Cargo.toml Updates dependencies and removes release profile LTO configuration.
.github/workflows/release.yml Revises release workflow to support semantic version tags and packaging.
.github/workflows/base.yml Adjusts workflow triggers for the release job.
.github/install-dep/action.yml Updates platform condition checks using broader matching.
Files not reviewed (1)
  • monobean/resources/gtk/hello_page.ui: Language not supported
Comments suppressed due to low confidence (2)

monobean/Cargo.toml:47

  • Removal of the 'lto = "fat"' configuration in the release profile may lead to less optimized builds; please confirm if this change is intentional.
-[profile.release]

.github/workflows/release.yml:52

  • [nitpick] Ensure consistency in naming conventions for the packaged binaries across platforms; the use of double underscores here contrasts with naming in other steps.
mv monobean/target/build/monobean__${{ matrix.arch }}.deb ${{ github.workspace }}/Monobean_${{ github.ref_name }}_${{ matrix.target }}.deb

Comment thread monobean/src/main.rs
Comment thread monobean/src/application.rs
Comment thread .github/install-dep/action.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@genedna
genedna merged commit 04e3a3b into gitmono-dev:main May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants