Skip to content

ci: pin Homebrew version in CI runs#51555

Merged
jkleinsc merged 1 commit into
40-x-yfrom
manual-bp/nmggithub/pr/51543/branch/40-x-y
May 8, 2026
Merged

ci: pin Homebrew version in CI runs#51555
jkleinsc merged 1 commit into
40-x-yfrom
manual-bp/nmggithub/pr/51543/branch/40-x-y

Conversation

@nmggithub

Copy link
Copy Markdown
Contributor

Backport of #51543.

See that PR for details.

Notes: none.

* feat: pin Homebrew version in CI runs

* feat: validate Homebrew asset before installing

* fix: fix sha256 verificaton

* fix: inline sha256 check

* fix: remove GitHub CLI integrity check
@nmggithub nmggithub requested a review from a team as a code owner May 7, 2026 21:23
@trop trop Bot added 40-x-y backport This is a backport PR semver/none labels May 7, 2026
@trop trop Bot mentioned this pull request May 7, 2026
1 task

@github-actions github-actions 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.

Hello @nmggithub! It looks like this pull request touches one of our dependency or CI files, and per our contribution policy we do not accept these types of changes in PRs.

To move this PR forward, please:

  1. Revert the dependency/CI file changes from your branch. (e.g. yarn.lock, .yarn/, .yarnrc.yml, .github/workflows/, .github/actions/)
  2. Ensure your branch allows maintainer commits so a maintainer can push the necessary dependency changes on your behalf.
  3. Leave a comment letting reviewers know the dependency change is still needed.
For maintainers

To land this PR, push a verified commit to the contributor's branch with the required dependency/CI changes, then dismiss this review.

@nikwen

nikwen commented May 7, 2026

Copy link
Copy Markdown
Member

I wonder why that workflow triggered for you. I believe it shouldn't have.

@Android-studio61

This comment was marked as spam.

@jkleinsc

jkleinsc commented May 8, 2026

Copy link
Copy Markdown
Member

Merging as CI failure unrelated to PR change.

@jkleinsc jkleinsc merged commit 08f4bb8 into 40-x-y May 8, 2026
123 of 126 checks passed
@jkleinsc jkleinsc deleted the manual-bp/nmggithub/pr/51543/branch/40-x-y branch May 8, 2026 13:55
@release-clerk

release-clerk Bot commented May 8, 2026

Copy link
Copy Markdown

No Release Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants