Skip to content

Remove windows-2019 from workflows as the image deprecation has begun on 2025-06-01 #343

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aparnajyothi-y
Copy link
Contributor

Removing the windows-2019 from workflows, as it the deprication started on 2026-06-01 and will be fully unsupported by 2025-06-30 as per actions/runner-images#12045.

@Copilot Copilot AI review requested due to automatic review settings June 9, 2025 13:30
@aparnajyothi-y aparnajyothi-y requested a review from a team as a code owner June 9, 2025 13:30
Copy link

@Copilot 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

Removes deprecated Windows 2019 runner images and replaces them with Windows 2022 across the Python package build workflow.

  • Update default inputs.platforms to use windows-2022 instead of windows-2019
  • Update the PowerShell matrix generation to reflect the new windows-2022 labels
Comments suppressed due to low confidence (1)

.github/workflows/build-python-packages.yml:47

  • The default matrix string uses macos-13 without an architecture suffix, whereas the inputs default lists macos-13_x64. Align these names to prevent confusion and ensure they match runner labels.
[String[]]$configurations = "${{ inputs.platforms || 'ubuntu-22.04,ubuntu-22.04_arm64,ubuntu-24.04,ubuntu-24.04_arm64,macos-13,macos-14_arm64,windows-2022_x64,windows-2022_x86,windows-11_arm64' }}".Split(",").Trim()

@@ -20,7 +20,7 @@ on:
PLATFORMS:
description: 'Platforms for execution in "os" or "os_arch" format (arch is "x64" by default)'
required: true
default: 'ubuntu-22.04,ubuntu-22.04_arm64,ubuntu-24.04,ubuntu-24.04_arm64,macos-13_x64,macos-14_arm64,windows-2019_x64,windows-2019_x86,windows-11_arm64'
default: 'ubuntu-22.04,ubuntu-22.04_arm64,ubuntu-24.04,ubuntu-24.04_arm64,macos-13_x64,macos-14_arm64,windows-2022_x64,windows-2022_x86,windows-11_arm64'
Copy link
Preview

Copilot AI Jun 9, 2025

Choose a reason for hiding this comment

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

The platforms list is duplicated between the workflow inputs and the PowerShell matrix script. Consider extracting this list into a single YAML anchor or environment variable to ensure consistency and reduce the chance of drift.

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.

1 participant