Skip to content

Update .NET SDK to v.10.0.300#42

Merged
CharlesFarris merged 3 commits into
mainfrom
update-net-sdk
Jun 3, 2026
Merged

Update .NET SDK to v.10.0.300#42
CharlesFarris merged 3 commits into
mainfrom
update-net-sdk

Conversation

@CharlesFarris

Copy link
Copy Markdown
Contributor
  • Fix code analysis issues

Copilot AI review requested due to automatic review settings June 3, 2026 03:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository’s pinned .NET SDK feature band and adjusts NUnit test assertions to address analyzer/code-analysis guidance while preserving existing test behavior.

Changes:

  • Bump global.json SDK version from 10.0.200 to 10.0.300.
  • Replace Assert.Multiple(() => { ... }) with using (Assert.EnterMultipleScope()) { ... } across several NUnit test files.

Reviewed changes

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

Show a summary per file
File Description
src/Monads.Tests/OptionExtensionsTests.cs Uses Assert.EnterMultipleScope() for grouped assertions in TrySome_* tests.
src/Errors.Tests/ValueErrorExtensionsTests.cs Switches to Assert.EnterMultipleScope() for assertions.
src/Errors.Tests/ValidationErrorExtensionTests.cs Switches grouped assertions to Assert.EnterMultipleScope().
src/Errors.Tests/ExceptionErrorExtensionsTests.cs Switches to Assert.EnterMultipleScope() for assertions.
src/Common.Tests/PipeExtensionsTests.cs Uses Assert.EnterMultipleScope() for grouped assertions in pipe-related tests.
src/Common.Tests/FuncExtensionsTests.cs Uses Assert.EnterMultipleScope() in multiple tests to group related assertions.
global.json Pins the SDK to 10.0.300 with rollForward: latestPatch.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@CharlesFarris CharlesFarris self-assigned this Jun 3, 2026
@CharlesFarris CharlesFarris marked this pull request as ready for review June 3, 2026 03:31
Copilot AI review requested due to automatic review settings June 3, 2026 03:31

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

@CharlesFarris CharlesFarris merged commit 440cb82 into main Jun 3, 2026
4 checks passed
@CharlesFarris CharlesFarris deleted the update-net-sdk branch June 3, 2026 03:34
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