Skip to content

fix: Remove unused local build scripts - #497

Open
DirkDoes wants to merge 1 commit into
mainfrom
fix/remove-unused-build-scripts
Open

DirkDoes wants to merge 1 commit into
mainfrom
fix/remove-unused-build-scripts

Conversation

@DirkDoes

@DirkDoes DirkDoes commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

Summary

  • remove obsolete root-level build scripts
  • keep the macOS release script used by GitHub Actions

Verification

  • confirmed no repository references remain to the deleted scripts
  • confirmed the release workflow still invokes macos/release-macos.sh

Summary by CodeRabbit

  • Chores
    • Removed the bundled build shortcuts for Windows, Linux (including ARM64), and macOS. These build workflows are no longer available through the project’s previous shortcuts. The changes affect build and packaging workflows; no changes to app features are described.

@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 9a327ac6-d0bf-43b2-ab54-eea1fe3a754a

📥 Commits

Reviewing files that changed from the base of the PR and between cf5f3ef and 22b1311.

📒 Files selected for processing (7)
  • build-everything.bat
  • build-linux-arm64.bat
  • build-linux-arm64.sh
  • build-linux.bat
  • build-linux.sh
  • build-mac.sh
  • build-win.bat
💤 Files with no reviewable changes (7)
  • build-everything.bat
  • build-mac.sh
  • build-linux-arm64.bat
  • build-win.bat
  • build-linux.sh
  • build-linux-arm64.sh
  • build-linux.bat

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

Seven build scripts were deleted. These included platform-specific publish commands and an aggregate Windows batch script that invoked Windows and Linux build scripts.

Changes

Build Script Removal

Layer / File(s) Summary
Remove build wrappers
build-everything.bat, build-win.bat, build-linux.bat, build-linux.sh, build-linux-arm64.bat, build-linux-arm64.sh, build-mac.sh
The aggregate batch script and platform build wrappers were removed. The wrappers included Windows, Linux x64, Linux ARM64, and macOS build commands.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to 22b13

The supported build and release workflows remain available. No actionable risk from the script removal was established.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: removal of obsolete local build scripts.
Description check ✅ Passed The description explains the purpose, changed files, and verification steps. It does not use the repository template headings and does not include a related issue link or completed checklist, but it p…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

A rabbit checks the build scripts’ trail.
The Windows batch file is gone.
Linux wrappers leave no trace.
The Mac shell script has hopped away.
I nibble clover and close the file.

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

@DirkDoes DirkDoes changed the title Remove unused local build scripts fix: Remove unused local build scripts Sep 28, 2026
@DirkDoes
DirkDoes marked this pull request as ready for review September 28, 2026 20:10

This branch has not been deployed

No deployments
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