Skip to content

ci: test Homebrew installation lifecycle on labeled PRs - #2732

Closed
fengmk2 wants to merge 14 commits into
mainfrom
ci/homebrew-install-e2e
Closed

fengmk2 wants to merge 14 commits into
mainfrom
ci/homebrew-install-e2e

Conversation

@fengmk2

@fengmk2 fengmk2 commented Sep 17, 2026

Copy link
Copy Markdown
Member

Homebrew installations need end-to-end coverage for the code under review. Add a macOS ARM64 job that runs when a pull request has the test: install-e2e label. Later pushes rerun it while the label remains.

The job adapts the current official formula to build the tested commit and its pinned upstream resources. It checks first-run setup, bundled commands without npm access, saved preferences, diagnostics, and removal. Two formula revisions exercise brew upgrade and brew cleanup, including existing Bash command caches and saved shim paths.

The bundled formatter check exposes a separate formula issue: --no-optional removes Oxfmt's required native binding. The Homebrew formula needs a packaging fix for this check to pass.

Failures retain the generated formulae, source revisions, command output, and Homebrew build logs as artifacts.

To run the checks locally, use a disposable Mac with Homebrew and Node.js:

node .github/scripts/test-homebrew.mjs /absolute/path/to/homebrew-e2e-results

@fengmk2
fengmk2 added this pull request to stack #2733 September 17, 2026 09:07
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 17, 2026

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://ci-homebrew-install-e2e-viteplus-dev.voidzero-docs.workers.dev (commit cdc2219)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://0d3d850c-viteplus-dev.voidzero-docs.workers.dev cdc2219 2026-09-17T14:09:24.833Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://fef86d09-viteplus-dev.voidzero-docs.workers.dev 290daf8 2026-09-17T09:16:47.262Z Visit the dashboard ↗
  • Build: In progress 🔵

View logs ↗
76ee5f6 2026-09-17T09:14:24.702Z View logs ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://6464b404-viteplus-dev.voidzero-docs.workers.dev 8992d25 2026-09-17T09:12:52.188Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://02e63eda-viteplus-dev.voidzero-docs.workers.dev 1664d6a 2026-09-17T09:08:56.921Z Visit the dashboard ↗

@fengmk2 fengmk2 added the test: install-e2e run vite install e2e test label Sep 17, 2026
@fengmk2
fengmk2 removed this pull request from stack #2733 September 17, 2026 14:47
Base automatically changed from fix/brew-self-setup-2719 to main September 17, 2026 14:47
@fengmk2 fengmk2 closed this Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test: install-e2e run vite install e2e test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant