Skip to content

Reduce boilerplate in e2e.yml using matrix strategy#3

Merged
Goooler merged 2 commits intomasterfrom
copilot/reduce-boilerplates-e2e-workflow
Mar 5, 2026
Merged

Reduce boilerplate in e2e.yml using matrix strategy#3
Goooler merged 2 commits intomasterfrom
copilot/reduce-boilerplates-e2e-workflow

Conversation

Copy link

Copilot AI commented Mar 4, 2026

Two nearly identical jobs (versions-macOS-15, versions-macOS-26) duplicated all steps, differing only in runner and Xcode versions.

Changes

  • Merged into single versions job using a 2D matrix (runner × xcode-version)
  • Shared versions (latest, latest-stable) expressed as cross-product across both runners
  • Runner-specific numeric versions added via include entries
strategy:
  matrix:
    runner: [macos-15, macos-26]
    xcode-version: [latest, latest-stable]
    include:
      # https://github.com/actions/runner-images/blob/main/images/macos/macos-15-Readme.md#xcode
      - runner: macos-15
        xcode-version: '16.2'
      - runner: macos-15
        xcode-version: '16.4'
      # https://github.com/actions/runner-images/blob/main/images/macos/macos-26-Readme.md#xcode
      - runner: macos-26
        xcode-version: '26.2'
      - runner: macos-26
        xcode-version: '26.3'
  fail-fast: false

All 8 original test combinations are preserved.

Co-authored-by: Goooler <10363352+Goooler@users.noreply.github.com>
Copilot AI changed the title [WIP] Reduce boilerplates in e2e.yml using matrix Reduce boilerplate in e2e.yml using matrix strategy Mar 4, 2026
@Goooler Goooler marked this pull request as ready for review March 4, 2026 11:58
@Goooler Goooler merged commit c97bb24 into master Mar 5, 2026
9 checks passed
@Goooler Goooler deleted the copilot/reduce-boilerplates-e2e-workflow branch March 5, 2026 03:13
@Goooler
Copy link
Owner

Goooler commented Mar 5, 2026

Addressed to maxim-lobanov#98.

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.

2 participants