Skip to content

feat: added shell scripts for build-linux and build-linux-arm64#304

Open
duderain wants to merge 2 commits into
TeamWheelWizard:mainfrom
duderain:main
Open

feat: added shell scripts for build-linux and build-linux-arm64#304
duderain wants to merge 2 commits into
TeamWheelWizard:mainfrom
duderain:main

Conversation

@duderain

@duderain duderain commented Jul 17, 2026

Copy link
Copy Markdown

Purpose of this PR:

Adds Shell scripts necessary to compile the targets linux-x64 and linux-arm64 on non-Windows platforms.

How to Test:

Run build-linux.sh and build-linux-arm64.sh to check the script runs properly.

What Has Been Changed:

build-linux.sh and build-linux-arm64.sh have been added.

Related Issue Link:

None.

Checklist before merging

  • You have created relevant tests

Summary by CodeRabbit

  • New Features
    • Added build options for Linux ARM64 and Linux x64.
    • Builds now produce optimized, self-contained single-file executables with native library support and compression.

@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Added Bash scripts to publish compressed, self-contained single-file Release binaries for Linux x64 and arm64 architectures.

Changes

Linux publishing

Layer / File(s) Summary
Linux release publishing scripts
build-linux.sh, build-linux-arm64.sh
Adds architecture-specific dotnet publish commands for self-contained, single-file Linux binaries with native library inclusion and compression enabled.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Poem

I’m a rabbit with builds in my burrow tonight,
Bundling Linux binaries, tidy and light.
X64 hops left, ARM64 hops right,
Single-file packages take flight!
Compressed and ready—what a wonderful sight.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title is concise and accurately describes the added Linux build scripts.
Description check ✅ Passed The description matches the required template and includes purpose, testing, changes, related issue, and checklist.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@build-linux.sh`:
- Around line 1-2: The publish commands in build-linux.sh (lines 1-2) and
build-linux-arm64.sh (lines 1-2) must target WheelWizard/WheelWizard.csproj and
write to publish/linux-x64 and publish/linux-arm64 respectively. Align each
script’s runtime and output path with the corresponding release workflow
contract.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9cde136d-e1d8-44ca-a1d2-be257184138c

📥 Commits

Reviewing files that changed from the base of the PR and between 4137cd3 and 92ede96.

📒 Files selected for processing (2)
  • build-linux-arm64.sh
  • build-linux.sh

Comment thread build-linux.sh
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