Skip to content

chore(deps): bump package and GitHub Actions versions#225

Merged
roryprimrose merged 1 commit into
mainfrom
chore/package-upgrades
Jul 20, 2026
Merged

chore(deps): bump package and GitHub Actions versions#225
roryprimrose merged 1 commit into
mainfrom
chore/package-upgrades

Conversation

@roryprimrose

Copy link
Copy Markdown
Owner

Consolidates the currently-open dependabot dependency PRs, applied and verified locally.

Included

Why #222 and #223 were combined

Microsoft.Extensions.DependencyInjection 10.0.10 has a transitive dependency requiring Microsoft.Bcl.AsyncInterfaces >= 10.0.10. With Central Package Management (CentralPackageTransitivePinningEnabled), bumping only one of these two centrally-pinned versions produces an NU1109 package downgrade error at restore. That's why dependabot's #223 and #222 each failed CI individually — they need to land together, which is confirmed by a clean local dotnet restore/build/test here.

Excluded: NSubstitute 5.3.0 → 6.0.0 (#221)

Left out intentionally. NSubstitute 6.0.0 enabled nullable reference annotations on Substitute.For/ForPartsOf/Arg.Is, which regressed support for legitimate null constructor arguments and matchers. This is an acknowledged upstream bug (not a deliberate breaking change): nsubstitute/NSubstitute#978, related #973, with an unmerged/contested fix in #976. Upgrading now would also risk breaking consumers' own NSubstitute-based test code if they upgrade in lockstep. Revisit once a fixed NSubstitute version ships.

Verification

  • dotnet build Neovolve.Streamline.slnx — 0 warnings, 0 errors
  • dotnet test Neovolve.Streamline.slnx — 161/161 passed (net472, net8.0, net9.0, net10.0)

- Microsoft.Bcl.AsyncInterfaces 10.0.9 -> 10.0.10
- Microsoft.Extensions.DependencyInjection 10.0.9 -> 10.0.10
- GitVersion.MsBuild 6.7.0 -> 6.8.2
- actions/setup-dotnet v5 -> v6
- actions/dependency-review-action v4 -> v5

Microsoft.Extensions.DependencyInjection 10.0.10 requires
Microsoft.Bcl.AsyncInterfaces >= 10.0.10 transitively, so these two
must be bumped together to avoid an NU1109 package downgrade error
under Central Package Management (this is why dependabot PRs #223 and
#222 individually failed CI).

NSubstitute 5.3.0 -> 6.0.0 (PR #221) is intentionally excluded: 6.0.0
enables nullable annotations on Substitute.For/ForPartsOf/Arg.Is that
regressed support for legitimate null constructor arguments/matches,
tracked upstream as nsubstitute/NSubstitute#978 and #973 with an
unmerged, contested fix (#976). Revisit once that lands.
@github-actions

Copy link
Copy Markdown

Summary - Code Coverage

Summary
Generated on: 7/20/2026 - 3:02:22 AM
Parser: Cobertura
Assemblies: 2
Classes: 10
Files: 9
Line coverage: 90.4% (237 of 262)
Covered lines: 237
Uncovered lines: 25
Coverable lines: 262
Total lines: 1025
Branch coverage: 80.7% (105 of 130)
Covered branches: 105
Total branches: 130
Method coverage: Feature is only available for sponsors
Tag: 485_29711033234

Coverage

Neovolve.Streamline - 89.4%
Name Line Branch
Neovolve.Streamline 89.4% 80.1%
Neovolve.Streamline.Tests 100%
Neovolve.Streamline.TestsBase 92.6% 83.8%
Neovolve.Streamline.TestsBase.ServiceProviderBridge 40% 25%
Neovolve.Streamline.TestsInternal 100%
Neovolve.Streamline.NSubstitute - 100%
Name Line Branch
Neovolve.Streamline.NSubstitute 100% 100%
NSubstitute.Tests 100%
NSubstitute.TestsInternal 100%
NSubstitute.TestsPartOf 100%
NSubstitute.TestsPartOfInternal 100%
NSubstitute.TestsSubstituteOf 100% 100%
NSubstitute.TestsSubstituteOfInternal 100% 100%

@roryprimrose
roryprimrose merged commit 9eedf7d into main Jul 20, 2026
10 checks passed
@roryprimrose
roryprimrose deleted the chore/package-upgrades branch July 20, 2026 03:02
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