Skip to content

Remove committed macOS binary and ignore build output - #126

Merged
thibauult merged 1 commit into
openhue:mainfrom
makoshark:remove-macos-binary
Oct 27, 2026
Merged

thibauult merged 1 commit into
openhue:mainfrom
makoshark:remove-macos-binary

Conversation

@makoshark

@makoshark makoshark commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

openhue/openhue-cli is a 13 MB Mach-O arm64 executable committed in f346b99 (#102), presumably a stray go build output. It is not referenced by the build and fails with exec format error on Linux, which is how I noticed it after cloning. This PR removes it and adds the default go build output name to .gitignore.

Summary by CodeRabbit

  • Chores
    • Updated repository housekeeping so generated command-line build output is excluded from version control. This change affects how build artifacts are handled during development and does not alter the app’s features or behavior for end users. No changes were made to public interfaces or functionality.

openhue/openhue-cli is a 13 MB Mach-O arm64 executable that was committed
in f346b99 (openhue#102), presumably a stray `go build` output. On Linux it fails
with "exec format error". Remove it and ignore the default build name.
@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f4820ee7-9581-49e7-b96a-9924016be99c
Commits

Reviewing files that changed from the base of the PR and between 08e940a and adb5a7e.

Files selected for processing: 2
  • .gitignore
  • openhue/openhue-cli

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

Walkthrough

The .gitignore now excludes the openhue-cli Go build output.

Changes: Go build output ignore

Layer / File(s) Summary
Add the build output ignore rule
.gitignore
Adds a comment and an ignore rule for openhue-cli.

Priority: Low

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

Change: Other

Merge Risk: Minimal · up to adb5a

The cleanup removes the committed binary and prevents the default build output from being recommitted.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and accurately describes the two main changes: removing the committed macOS binary and ignoring the build output.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within survi…
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.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@makoshark
makoshark marked this pull request as ready for review October 27, 2026 20:04

@thibauult thibauult 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.

LGTM 👍🏻

@thibauult
thibauult merged commit fc052c3 into openhue:main Oct 27, 2026
2 checks passed
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.

2 participants