Skip to content

chore: add open-pencil brew formula - #1302

Merged
shunkakinoki merged 1 commit into
mainfrom
chore/add-open-pencil
Mar 30, 2026
Merged

chore: add open-pencil brew formula#1302
shunkakinoki merged 1 commit into
mainfrom
chore/add-open-pencil

Conversation

@shunkakinoki

@shunkakinoki shunkakinoki commented Mar 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add open-pencil/tap tap and open-pencil brew formula to homebrew config

Summary by cubic

Adds the open-pencil/tap Homebrew tap and includes the open-pencil formula in the nix-darwin Homebrew config. This enables automatic install of Open Pencil during system setup.

Written for commit e0ead4f. Summary will update on new commits.

Entire-Checkpoint: d2b1945b5f9c
Copilot AI review requested due to automatic review settings March 30, 2026 16:52
@mesa-dot-dev

mesa-dot-dev Bot commented Mar 30, 2026

Copy link
Copy Markdown

You do not have enough credits to review this pull request. Please purchase more credits to continue.

@shunkakinoki
shunkakinoki merged commit 5c13c4f into main Mar 30, 2026
26 of 27 checks passed
@shunkakinoki
shunkakinoki deleted the chore/add-open-pencil branch March 30, 2026 16:52
@mesa-dot-dev

mesa-dot-dev Bot commented Mar 30, 2026

Copy link
Copy Markdown

Mesa Description

TL;DR

Add open-pencil/tap tap and open-pencil brew formula to homebrew config

What changed?

File summaries are not available to provide a detailed list of changes.

Description generated by Mesa. Update settings

@coderabbitai

coderabbitai Bot commented Mar 30, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 44abc47b-8786-4a80-9d31-29b77f3bcef6

📥 Commits

Reviewing files that changed from the base of the PR and between 3a6dcff and e0ead4f.

📒 Files selected for processing (1)
  • nix-darwin/config/homebrew.nix

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Summary by CodeRabbit

  • New Features
    • Added support for installing the open-pencil tool via Homebrew package manager

Walkthrough

Added the open-pencil/tap Homebrew tap and the open-pencil formula to the nix-darwin Homebrew configuration, enabling installation of the open-pencil tool during system activation.

Changes

Cohort / File(s) Summary
Homebrew Configuration
nix-darwin/config/homebrew.nix
Added open-pencil/tap to the homebrew.taps list and open-pencil to the homebrew.brews list.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A pencil so fine in the tap, so neat,
Now brewing with care in the rabbit's retreat,
One more tool, one more line, the config's complete,
Drawing dreams in the Nix, oh what a treat! ✏️

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/add-open-pencil

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.

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Copilot AI 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.

Pull request overview

Adds the open-pencil Homebrew formula to the nix-darwin Homebrew configuration by introducing the required tap and installing the formula via homebrew.brews.

Changes:

  • Add open-pencil/tap to the configured Homebrew taps
  • Add open-pencil to the installed Homebrew formula list

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

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request adds the 'open-pencil' tap and package to the Homebrew configuration in nix-darwin/config/homebrew.nix. Feedback was provided to maintain alphabetical sorting in both the taps and brews lists to improve maintainability and prevent duplicates.

Comment on lines 21 to 26
"paradigmxyz/brew"
"entireio/tap"
"manaflow-ai/cmux"
"open-pencil/tap"
"sst/tap"
"steipete/tap"

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.

medium

To improve maintainability, it's best to keep this list of taps sorted alphabetically. This makes it easier to find taps and see what's already included. The section of the list in this change is not sorted.

      "entireio/tap"
      "manaflow-ai/cmux"
      "open-pencil/tap"
      "paradigmxyz/brew"
      "sst/tap"
      "steipete/tap"

Comment on lines 47 to 50
"mlx-lm"
"open-pencil"
"opencode"
"ollama"

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.

medium

Similar to the taps list, this part of the brews list is not alphabetically sorted. To improve maintainability, please keep the list sorted. ollama should come before opencode and open-pencil.

      "mlx-lm"
      "ollama"
      "opencode"
      "open-pencil"

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