Skip to content

Use the latest version of runners#41

Merged
FanwangM merged 1 commit intomainfrom
fix_runners
Apr 24, 2026
Merged

Use the latest version of runners#41
FanwangM merged 1 commit intomainfrom
fix_runners

Conversation

@FanwangM
Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings April 24, 2026 14:38
@FanwangM FanwangM merged commit 31327a1 into main Apr 24, 2026
18 checks passed
@FanwangM FanwangM deleted the fix_runners branch April 24, 2026 14:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the GitHub Actions workflow to use the *-latest GitHub-hosted runner labels instead of pinned runner image versions.

Changes:

  • Switch matrix OS targets from pinned versions to ubuntu-latest, macos-latest, and windows-latest.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

fail-fast: false
matrix:
os: [ubuntu-22.04, macos-14, windows-2022]
os: [ubuntu-latest, macos-latest, windows-latest]
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using ubuntu-latest / macos-latest / windows-latest makes release builds non-deterministic because GitHub periodically repoints these labels to newer OS images. For wheel publishing this can unexpectedly change the toolchain / SDK and (especially on macOS/Windows) can change the effective minimum supported OS for produced wheels or break builds without any repo change. Consider pinning to specific runner images (e.g., ubuntu-22.04, macos-14, windows-2022) and updating them intentionally on a schedule, or add a clear justification and a plan to monitor runner label updates if you keep *-latest.

Copilot uses AI. Check for mistakes.
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