Skip to content

Allow using dashes (-) in tap names#21414

Merged
MikeMcQuaid merged 2 commits intoHomebrew:mainfrom
pkryger:allow-dashes-in-tap-names
Jan 17, 2026
Merged

Allow using dashes (-) in tap names#21414
MikeMcQuaid merged 2 commits intoHomebrew:mainfrom
pkryger:allow-dashes-in-tap-names

Conversation

@pkryger
Copy link
Copy Markdown
Contributor

@pkryger pkryger commented Jan 16, 2026

This allows for tap repositories to contain dashes anywhere in name not only in the homebrew- prefix.

This bring it on par with TestBot.HOMEBREW_TAP_REGEX (see https://github.com/Homebrew/brew/blob/110b93e/Library/Homebrew/test_bot.rb#L24) and Bundle.Dsl.Entry.HOMEBREW_TAP_ARGS_REGEX (see https://github.com/Homebrew/brew/blob/110b93e/Library/Homebrew/bundle/dsl.rb#L114).

I have stumbled upon it when wanted to add my formulae to runtime_cpu_detection_allowlist.json in pkryger/homebrew-emacsmacport-exp tap, and brew audit --strict has been not accepting my exceptions (see https://github.com/orgs/Homebrew/discussions/6656).

  • 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 for your changes? Here's an example.
  • Have you successfully run brew lgtm (style, typechecking and tests) with your changes locally?

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.

Seems reasonable, thanks!

@MikeMcQuaid MikeMcQuaid enabled auto-merge January 16, 2026 16:34
pkryger and others added 2 commits January 16, 2026 21:55
This allows for tap repositories to contain dashes anywhere in name not only in
the `homebrew-` prefix.

This bring it on par with `TestBot.HOMEBREW_TAP_REGEX` (see
https://github.com/Homebrew/brew/blob/110b93e/Library/Homebrew/test_bot.rb#L24)
and `Bundle.Dsl.Entry.HOMEBREW_TAP_ARGS_REGEX` (see
https://github.com/Homebrew/brew/blob/110b93e/Library/Homebrew/bundle/dsl.rb#L114).

I have stumbled upon it when wanted to add my formulae to
`runtime_cpu_detection_allowlist.json` in `pkryger/homebrew-emacsmacport-exp`
tap, and `brew audit --strict` has been not accepting my exceptions (see
https://github.com/orgs/Homebrew/discussions/6656).
@p-linnane p-linnane force-pushed the allow-dashes-in-tap-names branch from 9353b65 to cefe760 Compare January 17, 2026 05:59
@MikeMcQuaid MikeMcQuaid added this pull request to the merge queue Jan 17, 2026
Merged via the queue into Homebrew:main with commit 4c8621e Jan 17, 2026
36 checks passed
@pkryger pkryger deleted the allow-dashes-in-tap-names branch January 17, 2026 08:23
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