Repository navigation
ci(stryker): pin dotnet-stryker 5.0.0 and re-enable the PR gate - #381
Merged
Merged
Conversation
The PR trigger was disabled because every run crashed with TypeLoadException: Roslyn.Utilities.IObjectWritable. The cause was solution mode analysing examples/VB.*.vbproj, fixed by moving stryker-config.json into the test project (separate PR). Paths-filtered to src/, tests/ and this workflow. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Chris-Wolfgang
added a commit
that referenced
this pull request
Sep 27, 2026
…istically (#383) RunAsync_Action_CancellationToken_when_cancellation_is_requested_after_ action_started signalled "started" before the loop, so a fast runner could cancel before the first `Task.Delay(10).Wait()`, leaving that line uncovered on some runs. With the 100% test-assembly gate (#379) that made Stage 3 (macOS ARM64) fail at 97.9% on PR #381's run 36285791752 while Linux and Windows passed. The action now signals after a full pass of the loop body, so every line executes before the test cancels. Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
8 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This is the protected half of #281. Merge #378 first: it moves
stryker-config.jsonintotests/Wolfgang.TryPattern.Tests.Unit/, which removes the crash.dotnet-stryker4.16.0 → 5.0.0, the version repo-template's tool manifest pins and the one verified against this repo.mainandvNext, paths-filtered tosrc/**,tests/**and this workflow. That is ci(stryker): re-enable pull_request trigger once Buildalyzer supports Microsoft.CodeAnalysis 5.6+ #281's acceptance criterion. Stryker isn't a required check, so thepaths:filter can't leave PRs blocked.examples/VB.*.vbproj(TypeLoadException: IObjectWritable from VisualBasicCommandLineParser when solution contains a VB project (.NET 10 SDK) stryker-mutator/stryker-net#3666).Verification
With #378's config in place, plain
dotnet stryker(5.0.0) fromtests/Wolfgang.TryPattern.Tests.Unit(the Run step's existing(cd "$dir" && dotnet stryker)path) gave 147 mutants and a score of 92.66% in about 1.5 minutes, againstbreak: 60. actionlint and zizmor aren't installed locally; they run in CI.This touches
.github/workflows/, so it needs an admin-bypass merge.Closes #281
🤖 Generated with Claude Code