Skip to content

ci(release): conditionally exclude macos-15-intel from smoke test matrix#5311

Merged
avallete merged 1 commit into
developfrom
claude/fix-release-workflow-k5f85
May 20, 2026
Merged

ci(release): conditionally exclude macos-15-intel from smoke test matrix#5311
avallete merged 1 commit into
developfrom
claude/fix-release-workflow-k5f85

Conversation

@avallete
Copy link
Copy Markdown
Member

Refactor the smoke test job to conditionally exclude the slowest runner (macos-15-intel) from the test matrix on prerelease builds, rather than using a job-level if: condition.

Changes:

  • Moved the prerelease exclusion logic from a job-level if: condition into the strategy matrix itself using fromJSON()
  • The matrix now dynamically expands to either 3 runners (prerelease) or 4 runners (stable release) based on the inputs.prerelease flag
  • Updated comments to explain the rationale: macos-15-intel is the slowest leg and not available on Blacksmith, so excluding it from prerelease builds prevents it from blocking beta wall-clock time while stable releases still run the full matrix

Implementation details:

  • Uses GitHub Actions' fromJSON() to conditionally build the matrix list, since the matrix context is not available in job-level if: conditions
  • The conditional expression selects between two hardcoded runner arrays based on the prerelease flag
  • This approach ensures the job runs with the correct matrix expansion rather than skipping the entire job conditionally

https://claude.ai/code/session_01RDNmHeyREpf3ZBQLggK75q

…ix in job if

GitHub Actions evaluates job-level `if:` before matrix expansion, so the
`matrix.runner` context isn't available there — the workflow failed to
parse with "Unrecognized named-value: 'matrix'".

Build the matrix list via `fromJSON(inputs.prerelease && ... || ...)` so
macos-15-intel is excluded from the matrix on prereleases entirely, which
achieves the same goal (skip the slow Intel leg on PR/beta, keep it on
stable) without referencing `matrix` from a job condition.
@avallete avallete requested a review from a team as a code owner May 20, 2026 09:38
@avallete avallete enabled auto-merge (squash) May 20, 2026 09:39
@avallete avallete merged commit 6412213 into develop May 20, 2026
8 checks passed
@avallete avallete deleted the claude/fix-release-workflow-k5f85 branch May 20, 2026 09:45
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