Skip to content

Add Linuxbrew (Homebrew) install option for git-rain on Linux#4

Merged
bschellenberger2600 merged 1 commit into
mainfrom
cursor/linuxbrew-linux-options-9616
Apr 16, 2026
Merged

Add Linuxbrew (Homebrew) install option for git-rain on Linux#4
bschellenberger2600 merged 1 commit into
mainfrom
cursor/linuxbrew-linux-options-9616

Conversation

@bschellenberger2600
Copy link
Copy Markdown
Member

@bschellenberger2600 bschellenberger2600 commented Apr 16, 2026

Summary

git-fire already offered a Linux Homebrew (Linuxbrew) method in the install picker. git-rain did not. This change adds the same tap/install path for Linux users choosing git-rain.

Changes

  • Include brew in the ordered Linux method list for git-rain (after deb/rpm reordering, before manual).
  • Add rainLinuxBrew() and wire methodId === 'brew' in resolveCommand.

Verification

  • npm install and npm run build succeed locally.

Notes

Main was already up to date with origin/main at task start.

Open in Web Open in Cursor 

Summary by CodeRabbit

  • New Features

    • Introduced Linux Homebrew installation support for git-rain package resolution.
  • Improvements

    • Updated the default Linux installer method prioritization to better serve diverse installation scenarios and user preferences, now favoring standard package managers over manual installation paths.

Expose Homebrew/Linuxbrew alongside deb, rpm, and manual for the
git-rain Linux install picker, matching git-fire and macOS tap flow.

Co-authored-by: Ben Schellenberger <bschellenberger2600@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 16, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: fc39a896-366c-4c2f-abd9-f145ef7aec92

📥 Commits

Reviewing files that changed from the base of the PR and between c2e4d20 and 4a19c1f.

📒 Files selected for processing (1)
  • src/scripts/install-pickers.ts

📝 Walkthrough

Walkthrough

Linux installer method ordering for git-rain was updated to prioritize deb, rpm, brew, then manual. A new Linux Homebrew resolver function rainLinuxBrew was added to handle Homebrew-based installations and return appropriate tap and installation commands.

Changes

Cohort / File(s) Summary
Linux Installer Configuration
src/scripts/install-pickers.ts
Reordered method priority in the default pool (moved brew before manual), added rainLinuxBrew resolver function for Linuxbrew support, and updated command resolution logic to route methodId: 'brew' to the new resolver.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A brew of Linux magic unfolds,
Homebrew now finds its rightful place,
With methods reordered, the installer's mold,
Runs swift and smooth at its own pace!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/linuxbrew-linux-options-9616

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 and usage tips.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
git-fire-website 4a19c1f Commit Preview URL

Branch Preview URL
Apr 16 2026, 04:40 PM

@bschellenberger2600 bschellenberger2600 marked this pull request as ready for review April 16, 2026 16:45
@bschellenberger2600 bschellenberger2600 merged commit d3008fb into main Apr 16, 2026
3 of 4 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