Skip to content

Offer brew install bun in LLM guide, fix /pr commit collection#12

Merged
fetep merged 1 commit intomainfrom
petef/dev
Mar 18, 2026
Merged

Offer brew install bun in LLM guide, fix /pr commit collection#12
fetep merged 1 commit intomainfrom
petef/dev

Conversation

@fetep
Copy link
Copy Markdown
Owner

@fetep fetep commented Mar 18, 2026

Changes

  • docs: offer brew install bun in LLM guide, fix /pr commit collection

Copilot AI review requested due to automatic review settings March 18, 2026 23:54
@fetep fetep merged commit 79e86ce into main Mar 18, 2026
7 checks passed
@fetep fetep deleted the petef/dev branch March 18, 2026 23:55
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates installation guidance and refines the OpenCode /pr command workflow so PR descriptions only include truly-unmerged commits (even when prior PRs were squash-merged).

Changes:

  • README: detect Homebrew and offer installing Bun via brew install oven-sh/bun/bun when Bun is missing.
  • .opencode/command/pr.md: replace git log main..HEAD commit collection with git cherry + per-SHA git log to avoid including already-merged changes.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
README.md Adds Homebrew detection and a conditional Bun install path for the LLM install guide.
.opencode/command/pr.md Switches commit collection to git cherry to avoid false positives after squash merges.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

```

This outputs lines prefixed with `+` (unmerged) or `-` (already merged). Only use `+` lines. The format is `+ <sha> <subject>`.

Comment on lines +54 to +55
For each unmerged commit, get the full details:

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