Skip to content

Conversation

@danielmackay
Copy link
Member

@danielmackay danielmackay commented Dec 8, 2025

This pull request updates the project to target .NET 10 and upgrades several core dependencies to their latest major versions. It also removes redundant package references and disables a specific code style warning to allow for intentional use of backing fields. These changes ensure compatibility with .NET 10 and streamline project configuration.

Framework and SDK Upgrades

  • Updated TargetFramework to net10.0 in Directory.Build.props and removed old version references from ServiceDefaults.csproj and MigrationService.csproj. [1] [2] [3]
  • Updated .NET SDK version in global.json to 10.0.100.

Dependency Updates

  • Upgraded several key NuGet package versions to their 10.x releases in Directory.Packages.props, including Microsoft.AspNetCore.OpenApi, Microsoft.EntityFrameworkCore.Design, Microsoft.Extensions.Hosting, Microsoft.EntityFrameworkCore.Relational, Microsoft.Extensions.Http.Resilience, and Microsoft.AspNetCore.Mvc.Testing. [1] [2] [3]

Code Style and Build Configuration

  • Disabled the IDE0032 warning to allow intentional use of backing fields instead of auto-properties in Directory.Build.props.

Project Reference Cleanup

  • Removed unnecessary references to Microsoft.Extensions.DependencyInjection.Abstractions from WebApi.csproj and WebApi.IntegrationTests.csproj. [1] [2]

Copy link
Contributor

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 PR upgrades the project from .NET 9 to .NET 10, updating the SDK version, target framework, and Microsoft package dependencies to version 10.0.0. The changes also consolidate the target framework configuration by centralizing it in Directory.Build.props and removing redundant package references.

Key changes:

  • Upgraded .NET SDK from 9.0.300 to 10.0.100
  • Updated target framework from net9.0 to net10.0
  • Updated Microsoft packages from 9.x to 10.0.0 versions
  • Removed redundant Microsoft.Extensions.DependencyInjection.Abstractions package references
  • Added IDE0032 warning suppression for auto-property suggestions

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
global.json Updates SDK version to 10.0.100 for .NET 10
Directory.Build.props Changes target framework to net10.0 and adds IDE0032 NoWarn directive for intentional backing field usage
Directory.Packages.props Updates Microsoft package versions to 10.0.0 and removes Microsoft.Extensions.DependencyInjection.Abstractions
tools/MigrationService/MigrationService.csproj Removes explicit TargetFramework (now inherited from Directory.Build.props) and removes BOM character
src/ServiceDefaults/ServiceDefaults.csproj Removes explicit TargetFramework (now inherited from Directory.Build.props)
src/WebApi/WebApi.csproj Removes Microsoft.Extensions.DependencyInjection.Abstractions package reference
tests/WebApi.IntegrationTests/WebApi.IntegrationTests.csproj Removes Microsoft.Extensions.DependencyInjection.Abstractions package reference

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

Copy link
Member Author

@danielmackay danielmackay left a comment

Choose a reason for hiding this comment

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

LGTM

@danielmackay danielmackay merged commit 4887f9d into main Dec 8, 2025
2 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