Skip to content

fix: move open-pencil from brews to casks - #1327

Merged
shunkakinoki merged 1 commit into
mainfrom
fix/open-pencil-brew-cask
Apr 2, 2026
Merged

fix: move open-pencil from brews to casks#1327
shunkakinoki merged 1 commit into
mainfrom
fix/open-pencil-brew-cask

Conversation

@shunkakinoki

@shunkakinoki shunkakinoki commented Apr 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • open-pencil/tap only provides a cask, not a formula
  • Moved open-pencil from brews to casks in homebrew.nix

Test plan

  • make build && make switch completes without brew bundle errors for open-pencil

Summary by cubic

Move open-pencil from brews to casks in homebrew.nix to match the upstream tap (cask-only). This prevents brew bundle errors and lets make build && make switch complete successfully.

Written for commit 53d4522. Summary will update on new commits.

open-pencil/tap only provides a cask, not a formula.
Copilot AI review requested due to automatic review settings April 2, 2026 03:13
@coderabbitai

coderabbitai Bot commented Apr 2, 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: f520ffc5-5b44-49db-9122-f4b0ce4c9089

📥 Commits

Reviewing files that changed from the base of the PR and between c313d21 and 53d4522.

📒 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

  • Chores
    • Updated open-pencil package management to use Homebrew cask instead of brew formula, changing how the package is installed and managed.

Walkthrough

This PR relocates the open-pencil package from Homebrew's brews list to its casks list in the nix-darwin configuration, changing the installation mechanism for this package while maintaining the same file.

Changes

Cohort / File(s) Summary
Homebrew Configuration
nix-darwin/config/homebrew.nix
Moved open-pencil from homebrew.brews to homebrew.casks, changing its installation type from a formula to a cask.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A pencil once brewed, now casks it instead,
Moving through lists with a hop and a thread,
From formula to cask, a package's new fate,
Darwin's config stays nimble and great!

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

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.

@mesa-dot-dev

mesa-dot-dev Bot commented Apr 2, 2026

Copy link
Copy Markdown

Mesa Description

TL;DR

Moved open-pencil from Homebrew brews to casks as it is only available as a cask.

What changed?

  • open-pencil entry moved from brews to casks in homebrew.nix

Description generated by Mesa. Update settings

@shunkakinoki
shunkakinoki merged commit 9f91a04 into main Apr 2, 2026
35 of 36 checks passed
@shunkakinoki
shunkakinoki deleted the fix/open-pencil-brew-cask branch April 2, 2026 03:14

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

Moves the open-pencil Homebrew dependency to the correct install type in the nix-darwin Homebrew configuration, aligning with open-pencil/tap providing a cask rather than a formula.

Changes:

  • Remove open-pencil from homebrew.brews
  • Add open-pencil to homebrew.casks

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

@mesa-dot-dev mesa-dot-dev Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Performed full review of c313d21...53d4522

Analysis

No significant architectural issues identified. This PR contains a straightforward categorization fix moving open-pencil from brews to casks, which aligns with Homebrew conventions and existing patterns in the codebase. The change is minimal, maintains alphabetical ordering, and has no ripple effects across other configuration files. Safe to merge.

Tip

Help

Slash Commands:

  • /review - Request a full code review
  • /review latest - Review only changes since the last review
  • /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
  • /help - Get help with Mesa commands and configuration options

0 files reviewed | 0 comments | Edit Agent SettingsRead Docs

@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

@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 moves the 'open-pencil' package within the Homebrew configuration in nix-darwin/config/homebrew.nix, likely transitioning it from a formula to a cask. I have no feedback to provide.

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