Skip to content

Upgrade GitHub Actions runner images to windows-2025#805

Open
Arlodotexe wants to merge 5 commits into
mainfrom
check,ci/runner,images,github,actions/improvement/windows-2025,upgrade,windows-sdks/5.15.2026,arlodotexe
Open

Upgrade GitHub Actions runner images to windows-2025#805
Arlodotexe wants to merge 5 commits into
mainfrom
check,ci/runner,images,github,actions/improvement/windows-2025,upgrade,windows-sdks/5.15.2026,arlodotexe

Conversation

@Arlodotexe
Copy link
Copy Markdown
Member

Background

PR #803 intended to fully upgrade us to the windows-2025 runners and has been reviewed by a frequent contributor, closed and merged into main.

Problem

This was surfaced by @Avid29 in the #community-toolkit - Targeting .NET 10 thread on Discord after bringing in the #803 work to unblock #791.

The broken CI problem only appears on #791 and not on main because surfacing it requires bumping our minimum msbuild version to at least 18.0 and it requires bumping the runner (v17.0 is not included on windows-2025).

Some small but obvious edits were missed and required a follow up PR to fix.

Solution

Corrections have been made. Small PR.

@Arlodotexe Arlodotexe self-assigned this May 15, 2026
@Arlodotexe Arlodotexe requested a review from Avid29 May 15, 2026 10:51
Avid29
Avid29 previously approved these changes May 15, 2026
@Avid29 Avid29 self-requested a review May 15, 2026 11:00
@Avid29 Avid29 dismissed their stale review May 15, 2026 11:00

Must resolve disc size issues first

@Arlodotexe
Copy link
Copy Markdown
Member Author

Arlodotexe commented May 15, 2026

Found the background info and original source of the bug for the disk space issue here.

The mistake was made on Oct 2nd 2025 under https://github.com/CommunityToolkit/Windows/pull/737/files.

Per the comments from @michael-hawker in the PR/Issue on this, we intended to keep the "large" runner wherever it was used, but the change here doesn't reflect that.

It looks like the bad change has persisted unnoticed since then. The fix here is now certain and should be simple, I'll push that up in a moment.

@Arlodotexe
Copy link
Copy Markdown
Member Author

Closing and re-opening to hopefully rekick CI

@Arlodotexe Arlodotexe closed this May 15, 2026
@Arlodotexe Arlodotexe reopened this May 15, 2026
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