Skip to content

Comments

protobuf: remove unneeded OS.mac?#233608

Merged
carlocab merged 1 commit intomainfrom
protobuf-macos-conditional
Aug 15, 2025
Merged

protobuf: remove unneeded OS.mac?#233608
carlocab merged 1 commit intomainfrom
protobuf-macos-conditional

Conversation

@carlocab
Copy link
Member

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

I missed this in #227169.

@carlocab carlocab added CI-skip-dependents Pass --skip-dependents to brew test-bot. CI-no-bottles Merge without publishing bottles labels Aug 15, 2025
@carlocab carlocab enabled auto-merge August 15, 2025 08:28
@carlocab carlocab added this pull request to the merge queue Aug 15, 2025
Merged via the queue into main with commit 81a9c06 Aug 15, 2025
34 checks passed
@carlocab carlocab deleted the protobuf-macos-conditional branch August 15, 2025 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-no-bottles Merge without publishing bottles CI-skip-dependents Pass --skip-dependents to brew test-bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants