Skip to content

Conversation

@TonyTroeff
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings December 11, 2025 09:19
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds .NET 8 support to the OneBitSoftware.Utilities.OperationResult library, enabling multi-targeting for both .NET 8.0 and .NET 9.0 frameworks.

  • Updated project files to target both net8.0 and net9.0 frameworks
  • Added framework-specific package references for Microsoft.Extensions.Logging.Abstractions
  • Updated CI/CD workflow to install and test against both .NET SDK versions
  • Incremented package version from 2.1.0 to 2.1.1

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
tests/OneBitSoftware.Utilities.OperationResultTests/OneBitSoftware.Utilities.OperationResultTests.csproj Changed from single-target (net9.0) to multi-target (net8.0;net9.0) configuration
src/OneBitSoftware.Utilities.OperationResult/OneBitSoftware.Utilities.OperationResult.csproj Updated to multi-target net8.0 and net9.0, added conditional package references for framework-specific logging abstractions versions, and bumped package version to 2.1.1
.github/workflows/pull-request-validation.yml Updated to install both .NET 8.x and 9.x SDKs and added configuration flag to test command for consistency

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

@SharePointRadi SharePointRadi merged commit 8aaaf11 into main Dec 11, 2025
7 checks passed
@TonyTroeff TonyTroeff deleted the troeff/add-support-for-dotnet8 branch December 11, 2025 13:43
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.

3 participants