Skip to content

Run tests against linux - #1314

Merged
GrahamTheCoder merged 5 commits into
icsharpcode:masterfrom
GrahamTheCoder:claude/wizardly-pasteur-e66wcm
Nov 11, 2026
Merged

GrahamTheCoder merged 5 commits into
icsharpcode:masterfrom
GrahamTheCoder:claude/wizardly-pasteur-e66wcm

Conversation

@GrahamTheCoder

@GrahamTheCoder GrahamTheCoder commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Test matrix is two jobs, each option once, differing only by OS and .NET version: windows-latest + .NET 10, and ubuntu-latest + .NET 8 (previous LTS).
  • dotnet-tests.yml no longer sets up msbuild or takes a vs-version. Both OSes build and test Tests/Tests.csproj for the matching framework (the slnf also builds the Web project, which needs node and which the tests don't use; the build job still covers the full slnf and the Vsix).
  • The net8 path reuses the existing update-testdata-targetframework.ps1 step.

For the reviewer

  • Job names changed (test-vs2022/test-vs2026 are now test-windows-latest/test-linux-previous); update branch protection if those were required checks.
  • Risk: Tests/TestData includes net48 projects loaded via MSBuildWorkspace. Hosted Windows runners still have Visual Studio installed, but it's no longer pinned. Needs the CI run to confirm.

Out of scope

Test performance seems worse than it used to be, but wasn't changed by this PR

🤖 Generated with Claude Code

https://claude.ai/code/session_019FEr8DKN7HXRmh5ejKjfRE

Gives OS x .NET version pairwise coverage. The reusable test workflow now
skips Windows-only msbuild setup and builds Tests.csproj directly on Linux
(the slnf pulls in the Web esproj, which the tests don't need).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019FEr8DKN7HXRmh5ejKjfRE
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019FEr8DKN7HXRmh5ejKjfRE
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019FEr8DKN7HXRmh5ejKjfRE
Drops setup-msbuild/vs-version and uses windows-latest/ubuntu-latest.
Both OSes build and test Tests.csproj for the matching framework.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019FEr8DKN7HXRmh5ejKjfRE
The projects also target net10.0, so the build needs the newest SDK (as before
the refactor). SDK 8 is only pinned for the test run. 8.0.0 is rejected by
newer hosts as an SDK version (feature bands start at 100).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019FEr8DKN7HXRmh5ejKjfRE
@GrahamTheCoder GrahamTheCoder changed the title Claude/wizardly pasteur e66wcm Run tests against linux Nov 11, 2026
@GrahamTheCoder
GrahamTheCoder merged commit 5daeea3 into icsharpcode:master Nov 11, 2026
3 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