-
-
Notifications
You must be signed in to change notification settings - Fork 363
feat(TargetFramework): support net10 #7100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this 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 adds support for .NET 10.0 target framework and refactors test project configurations for improved maintainability.
- Removes conditional versioning based on Visual Studio version, standardizing on .NET 10
- Adds net10.0 to the target frameworks list (net6.0, net7.0, net8.0, net9.0, net10.0)
- Centralizes test package references in test/Directory.Build.props
- Reorganizes solution file structure with better folder grouping
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| Version.props | Removed Visual Studio version conditions, set net10.0 as default target framework and added it to multi-targeting list |
| src/BootstrapBlazor/BootstrapBlazor.csproj | Removed conditional versioning, set version to 10.0.0 |
| test/Directory.Build.props | Centralized test package references (Microsoft.NET.Test.Sdk, xunit, coverlet), removed global Using directives |
| test/UnitTest/UnitTest.csproj | Removed duplicated test package references, added project-specific Using directives |
| test/UnitTest.Localization/UnitTest.Localization.csproj | Removed duplicated test package references, added project-specific Using directives |
| test/UnitTestDocs/UnitTestDocs.csproj | Removed duplicated test package references and property group, removed obsolete Using Remove directives |
| test/UniTest.Sass/UniTest.Sass.csproj | Cleaned up to minimal project file by removing package references and Using Remove directives |
| BootstrapBlazor.slnx | Reorganized file structure with dedicated folders for props files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7100 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 745 745
Lines 32548 32548
Branches 4510 4510
=========================================
Hits 32548 32548
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Link issues
fixes #7099
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
New Features: