Skip to content

Comments

CI: remove deprecated windows-2019 usage#19642

Merged
redsun82 merged 2 commits intomainfrom
redsun82/rm-windows-2019
Jun 2, 2025
Merged

CI: remove deprecated windows-2019 usage#19642
redsun82 merged 2 commits intomainfrom
redsun82/rm-windows-2019

Conversation

@redsun82
Copy link
Contributor

@redsun82 redsun82 commented Jun 2, 2025

The rip-unzip build workflow is using slightly older runners to ensure being runnable on supported platforms, though admittedly I don't know whether that can be an issue on windows.

Copilot AI review requested due to automatic review settings June 2, 2025 15:26
@redsun82 redsun82 requested a review from a team as a code owner June 2, 2025 15:26
@redsun82 redsun82 force-pushed the redsun82/rm-windows-2019 branch from 11e8bad to b1afa66 Compare June 2, 2025 15:26
Copy link
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

This PR removes deprecated Windows 2019 runners and updates all OS versions in the build-ripunzip workflow.

  • Replace windows-2019 with windows-latest in the C# QL test workflow
  • Update build-ripunzip runners from specific older versions to newer releases

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/csharp-qltest.yml Switched matrix OS from windows-2019 to windows-latest
.github/workflows/build-ripunzip.yml Updated matrix OS entries to ubuntu-24.04, macos-15, windows-2025
Comments suppressed due to low confidence (1)

.github/workflows/build-ripunzip.yml:20

  • The runner labels ubuntu-24.04, macos-15, and windows-2025 are not currently supported by GitHub Actions. Please use valid GitHub-hosted runners such as ubuntu-latest, macos-latest, or a specific supported version like windows-2022.
os: [ubuntu-24.04, macos-15, windows-2025]

@redsun82
Copy link
Contributor Author

redsun82 commented Jun 2, 2025

Successful run of the ripunzip workflow: https://github.com/github/codeql/actions/runs/15396339049

@redsun82 redsun82 merged commit 51ef76a into main Jun 2, 2025
12 checks passed
@redsun82 redsun82 deleted the redsun82/rm-windows-2019 branch June 2, 2025 16:05
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