Skip to content

feat(ci): add x86_64-apple-darwin to test and CLI E2E test matrices#671

Draft
fengmk2 wants to merge 3 commits intomainfrom
x86_64-apple-darwin-ci
Draft

feat(ci): add x86_64-apple-darwin to test and CLI E2E test matrices#671
fengmk2 wants to merge 3 commits intomainfrom
x86_64-apple-darwin-ci

Conversation

@fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Mar 2, 2026

Add Intel Mac (macos-15-intel) runner to both the test and
cli-e2e-test CI jobs. Update the download-rolldown-binaries action
to map target triples to npm binding package names so the correct
darwin-x64 binary is fetched for x86_64-apple-darwin.

@fengmk2 fengmk2 self-assigned this Mar 2, 2026
Copy link
Member Author

fengmk2 commented Mar 2, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

Add Intel Mac (macos-15-intel) runner to both the `test` and
`cli-e2e-test` CI jobs. Update the download-rolldown-binaries action
to map target triples to npm binding package names so the correct
darwin-x64 binary is fetched for x86_64-apple-darwin.
@fengmk2 fengmk2 force-pushed the x86_64-apple-darwin-ci branch from 67b9502 to 8cc880d Compare March 2, 2026 15:12
fengmk2 added 2 commits March 2, 2026 23:16
Use runner.os + runner.arch instead of inputs.target to select the
rolldown binding. The binding runs under the host Node process, so it
must match the host architecture, not the cross-compilation target.
The action now derives the platform from runner.os/runner.arch, so
the target input is unused. Remove it and clean up the two call sites
that were still passing it. Add an error fallthrough for unrecognized
platform combinations.
Copy link
Member Author

fengmk2 commented Mar 2, 2026

image.png

tsdown sea not support macos-15-intel

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.

1 participant