Skip to content

docs: add brew trust step for Homebrew 6 - #45

Merged
leonid-shevtsov merged 1 commit into
mainfrom
docs/mt-23212-brew-trust-readme
Jul 28, 2026
Merged

docs: add brew trust step for Homebrew 6#45
leonid-shevtsov merged 1 commit into
mainfrom
docs/mt-23212-brew-trust-readme

Conversation

@leonid-shevtsov

@leonid-shevtsov leonid-shevtsov commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

What & why

Homebrew 6 requires trusting third-party formulae before install. Found while testing brew install from the mailtrap/local tap — README was missing the trust step.

  • Add brew trust --formula mailtrap/local/mailtrap-local to the Homebrew install block

How to test

No QA needed — docs-only.

Summary by CodeRabbit

  • Documentation
    • Updated the Homebrew Quick Start instructions to include a trust step, ensuring the formula installation flow works as documented.

Decisions:
- Homebrew 6 requires explicit formula trust before third-party tap installs

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 69de2c8b-12d5-4544-a6f9-09f9395a532f

📥 Commits

Reviewing files that changed from the base of the PR and between 6f57564 and 11f102b.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The README Homebrew quick-start instructions now add a formula trust command immediately after adding the mailtrap/local tap.

Changes

Homebrew documentation

Layer / File(s) Summary
Add Homebrew formula trust step
README.md
The Homebrew installation flow adds brew trust --formula mailtrap/local/mailtrap-local after the tap command.

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

Possibly related PRs

Suggested reviewers: sergeykorolev

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately summarizes the docs change to add the Homebrew trust step.
Description check ✅ Passed It includes the main motivation, change summary, and testing note; the missing images/GIFs section is non-critical for this docs-only PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

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.

@leonid-shevtsov
leonid-shevtsov merged commit e7f9b60 into main Jul 28, 2026
5 checks passed
@leonid-shevtsov
leonid-shevtsov deleted the docs/mt-23212-brew-trust-readme branch July 28, 2026 07:54
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.

3 participants