Skip to content

Add platform-specific packaging workflow#110

Merged
beNative merged 2 commits intomainfrom
codex/create-github-actions-build-workflow
Oct 17, 2025
Merged

Add platform-specific packaging workflow#110
beNative merged 2 commits intomainfrom
codex/create-github-actions-build-workflow

Conversation

@beNative
Copy link
Copy Markdown
Owner

Summary

  • replace the legacy CI workflow with a packaging-focused workflow that targets macOS, Windows (x64/ia32), and Linux (x64/arm64/armv7l)
  • add npm scripts for each platform so the workflow can invoke platform-aware electron-builder commands with the correct flags
  • configure caching, environment variables, and artifact handling to improve build stability across runners

Testing

  • not run (workflow change)

https://chatgpt.com/codex/tasks/task_e_68f2684d8aec8332ab0f61be2f64b940

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting

Comment on lines +104 to +108
- name: Install system dependencies (Linux)
if: runner.os == 'Linux'
run: |
sudo apt-get update
sudo apt-get install --no-install-recommends -y rpm libarchive-tools libfuse2
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Linux packaging job installs non‑existent libfuse2 on ubuntu‑latest

The new workflow unconditionally installs libfuse2 before packaging. ubuntu-latest runners now use Ubuntu 24.04 where the package was renamed to libfuse2t64, so apt-get install --no-install-recommends -y rpm libarchive-tools libfuse2 exits with “Unable to locate package libfuse2” and the entire Linux matrix (x64/arm64/armv7l) fails before any build occurs. Consider installing libfuse2t64 or falling back when libfuse2 is unavailable so the workflow runs on current runners.

Useful? React with 👍 / 👎.

@beNative beNative merged commit cb9faed into main Oct 17, 2025
6 checks passed
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.

1 participant