Skip to content

Reduce CMake CI matrix to active GitHub-hosted runner images#34

Merged
yanyiwu merged 2 commits intomasterfrom
copilot/review-actions-failure
Mar 13, 2026
Merged

Reduce CMake CI matrix to active GitHub-hosted runner images#34
yanyiwu merged 2 commits intomasterfrom
copilot/review-actions-failure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 13, 2026

PR #32 was reported as having failing Actions, but the underlying issue was not a test regression. The CMake workflow was carrying legacy matrix entries that were being cancelled on macos-13 and windows-2019, which made the run status noisy and misleading.

  • Root cause

    • The CMake workflow matrix included runner images that were not completing reliably:
      • macos-13
      • windows-2019
    • The remaining matrix jobs were already completing successfully, so the issue was in CI orchestration rather than the SimHash change itself.
  • Workflow change

    • Trimmed the OS matrix in .github/workflows/cmake.yml to keep only the actively passing runner set:
      • ubuntu-22.04
      • ubuntu-24.04
      • macos-14
      • windows-2022
  • Effect

    • Removes the cancelled legacy runner jobs from the workflow result.
    • Preserves cross-platform coverage across Linux, macOS, and Windows without changing build or test logic.
strategy:
  matrix:
    os:
      - ubuntu-22.04
      - ubuntu-24.04
      - macos-14
      - windows-2022

📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Co-authored-by: yanyiwu <2162645+yanyiwu@users.noreply.github.com>
Copilot AI changed the title [WIP] Investigate actions failure in PR 32 Reduce CMake CI matrix to active GitHub-hosted runner images Mar 13, 2026
Copilot AI requested a review from yanyiwu March 13, 2026 08:02
@yanyiwu yanyiwu marked this pull request as ready for review March 13, 2026 09:27
@yanyiwu yanyiwu merged commit cd2aac6 into master Mar 13, 2026
64 checks passed
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