Skip to content

Fix test-bot links in documentation#21826

Merged
MikeMcQuaid merged 3 commits intoHomebrew:mainfrom
dnicolson:fix-test-bot-links
Mar 25, 2026
Merged

Fix test-bot links in documentation#21826
MikeMcQuaid merged 3 commits intoHomebrew:mainfrom
dnicolson:fix-test-bot-links

Conversation

@dnicolson
Copy link
Copy Markdown
Contributor


  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests (excluding integration tests) for your changes? Here's an example.
  • Have you successfully run brew lgtm (style, typechecking and tests) with your changes locally?

  • AI was used to generate or assist with generating this PR. Please specify below how you used AI to help you, and what steps you have taken to manually verify the changes.

Copilot AI review requested due to automatic review settings March 24, 2026 21:02
Copy link
Copy Markdown
Contributor

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 Homebrew’s documentation to point brew test-bot references at the correct source file in the Homebrew/brew repository, improving link accuracy for readers looking up implementation details.

Changes:

  • Update External-Commands.md to link to the in-repo dev-cmd/test-bot.rb file.
  • Update BrewTestBot.md to link to the same in-repo dev-cmd/test-bot.rb file instead of the external repository.

Reviewed changes

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

File Description
docs/External-Commands.md Fixes the brew test-bot example link to point at the in-repo command implementation.
docs/BrewTestBot.md Updates the test-bot.rb link to the in-repo implementation.

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

Copy link
Copy Markdown
Member

@ZhongRuoyu ZhongRuoyu left a comment

Choose a reason for hiding this comment

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

Good catch, thanks!

Co-authored-by: Ruoyu Zhong <zhongruoyu@outlook.com>
Copy link
Copy Markdown
Member

@MikeMcQuaid MikeMcQuaid left a comment

Choose a reason for hiding this comment

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

Thanks!

@MikeMcQuaid MikeMcQuaid enabled auto-merge March 25, 2026 08:34
@MikeMcQuaid MikeMcQuaid added this pull request to the merge queue Mar 25, 2026
Merged via the queue into Homebrew:main with commit b33ec59 Mar 25, 2026
57 of 58 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.

4 participants