Skip to content

Conversation

@gimlichael
Copy link
Member

This pull request updates the project to support .NET 10 and removes support for .NET 8 across the documentation, release notes, and build pipeline. It also upgrades dependencies and workflow actions to their latest compatible versions. The changes ensure the codebase is ready for modern .NET development and maintain compatibility with previous supported frameworks.

Platform and Target Framework Updates:

  • Updated all relevant target frameworks in .docfx/docfx.json from net9.0 to net10.0 to add support for .NET 10 and remove support for .NET 8. (.docfx/docfx.json: [1] [2] [3] [4]
  • Updated Dockerfile and DocFX build image versions to latest compatible releases. (.docfx/Dockerfile.docfx: .docfx/Dockerfile.docfxL1-R6)

Documentation and Release Notes:

  • Updated availability statements in documentation includes and release notes to reflect support for .NET 10 and removal of .NET 8. (.docfx/includes/availability-all.md: [1] .docfx/includes/availability-default.md: [2] .docfx/includes/availability-hybrid.md: [3] .docfx/includes/availability-modern.md: [4] .nuget/Cuemon.AspNetCore.App/PackageReleaseNotes.txt: [5] .nuget/Cuemon.AspNetCore.Authentication/PackageReleaseNotes.txt: [6] .nuget/Cuemon.AspNetCore.Mvc/PackageReleaseNotes.txt: [7]
  • Cleaned up older release notes and updated README files to clarify supported platforms and frameworks. (.nuget/Cuemon.AspNetCore.App/PackageReleaseNotes.txt: [1] .nuget/Cuemon.AspNetCore.App/README.md: [2] .nuget/Cuemon.AspNetCore.Authentication/PackageReleaseNotes.txt: [3] .nuget/Cuemon.AspNetCore.Authentication/README.md: [4]

Build and CI Pipeline Updates:

  • Upgraded workflow actions in .github/workflows/pipelines.yml to latest major versions (v3), including restore, build, pack, test, SonarCloud, CodeQL, and .NET installation steps. [1] [2] [3] [4] [5] [6] [7] [8]
  • Added explicit build steps for tests due to xUnit v3 requirements. [1] [2]

These updates ensure the project is aligned with the latest .NET ecosystem while maintaining a clean and up-to-date documentation and build pipeline.

gimlichael added 30 commits July 7, 2025 15:33
➖ remove support for net8.0 TFM
⬆️ bump dependencies
➖ remove explicit reference to System.Text.Json for TFM > net9.0
⬆️ bump dependencies
@gimlichael gimlichael requested a review from Copilot November 11, 2025 15:48
@gimlichael gimlichael self-assigned this Nov 11, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 11, 2025

Important

Review skipped

More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.

193 files out of 300 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch v10.0.0/launch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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 PR updates the project to .NET 10 (LTS) and removes support for .NET 8, aligning with a strategy to support only the latest LTS and STS .NET versions. It includes comprehensive updates to documentation, dependencies, build configurations, and test infrastructure to support .NET 10 while maintaining compatibility with .NET 9 (STS) and .NET Standard 2.0/2.1 where applicable.

Key Changes:

  • Added .NET 10 (LTS) support and removed .NET 8 (LTS) support across all target frameworks
  • Updated dependencies and GitHub Actions workflow to latest compatible versions
  • Migrated from xUnit v2 to xUnit v3, requiring explicit test build steps
  • Replaced conditional compilation directives from NET8_0_OR_GREATER to NET9_0_OR_GREATER
  • Updated lock implementations to use System.Threading.Lock for .NET 9+ while maintaining backward compatibility

Reviewed Changes

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

Show a summary per file
File Description
Test project files (*.csproj) Updated TargetFrameworks from net9.0;net8.0 to net10.0;net9.0
Test files (*.cs) Removed unused Xunit.Abstractions imports
Directory.Build.props Updated default target frameworks and added xUnit v3 package references
Directory.Packages.props Updated all package dependencies to latest versions for .NET 10
Source files (*.cs) Updated conditional compilation from NET8_0_OR_GREATER to NET9_0_OR_GREATER
Source files with locks Added conditional System.Threading.Lock usage for .NET 9+ with object fallback
Documentation/README files Updated availability statements and supported framework versions
Release notes Added version 10.0.0 entries documenting framework support changes
CHANGELOG.md Added entry for version 10.0.0 with breaking changes and dependency removals

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

@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

❌ Patch coverage is 83.33333% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.94%. Comparing base (c4ed5ad) to head (ef56bc5).

Files with missing lines Patch % Lines
src/Cuemon.Data/DatabaseWatcher.cs 0.00% 3 Missing ⚠️
...uemon.Extensions.Net/Http/SlimHttpClientFactory.cs 60.00% 2 Missing ⚠️
src/Cuemon.Core/Runtime/Watcher.cs 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
+ Coverage   79.86%   79.94%   +0.08%     
==========================================
  Files         595      595              
  Lines       18317    18289      -28     
  Branches     1870     1868       -2     
==========================================
- Hits        14629    14622       -7     
+ Misses       3618     3599      -19     
+ Partials       70       68       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sonarqubecloud
Copy link

@gimlichael gimlichael merged commit 90e8288 into main Nov 11, 2025
9 of 12 checks passed
@gimlichael gimlichael deleted the v10.0.0/launch branch November 11, 2025 17:09
@gimlichael gimlichael restored the v10.0.0/launch branch November 11, 2025 17:28
@gimlichael gimlichael deleted the v10.0.0/launch branch November 11, 2025 20:38
@gimlichael gimlichael restored the v10.0.0/launch branch November 11, 2025 20:44
@gimlichael gimlichael deleted the v10.0.0/launch branch November 12, 2025 00:26
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