Skip to content

feat: Update to dotnet10#24

Merged
danielpindur merged 3 commits intomasterfrom
dotnet10_update
Jan 19, 2026
Merged

feat: Update to dotnet10#24
danielpindur merged 3 commits intomasterfrom
dotnet10_update

Conversation

@danielpindur
Copy link
Owner

@danielpindur danielpindur commented Jan 19, 2026

Summary by CodeRabbit

  • Chores
    • Upgraded .NET target framework to 10.0 across projects and CI workflows.
    • Updated compiler/test SDK and test adapter packages to newer versions.
    • Minor internal code cleanups and formatting improvements (no behavioral changes).

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

📝 Walkthrough

Walkthrough

Updated projects and CI to target .NET 10.0; bumped Roslyn and test SDK package versions; added an enum-mapping override method to the Abstractions interface; introduced WellKnownTypes constants and updated generator symbol lookups; plus minor formatting and whitespace edits.

Changes

Cohort / File(s) Summary
CI/CD Workflows
.github/workflows/dotnet-ci.yml, .github/workflows/release-please.yml
Bumped .NET SDK usage to 10.0 (dotnet-version / dotnet-version: '10.0.x').
Example projects
examples/Example.Core/Example.Core.Api/Example.Core.Api.csproj, examples/Example.Core/Example.Core/Example.Core.csproj
Updated TargetFramework to net10.0; Example.Core.csproj replaced a PackageReference to Dango with two ProjectReferences.
Tests & Core project
test/Dango.Tests/Dango.Tests.csproj, src/Dango/Dango.csproj
Updated TargetFramework to net10.0; upgraded Microsoft.CodeAnalysis.CSharp (4.14.0 → 5.0.0) and test SDK/adapters.
Abstractions API
src/Dango.Abstractions/IDangoEnumMappingConfiguration.cs
Added WithOverrides(IReadOnlyDictionary<TSource, TDestination> overrides) to the interface.
Generator refactor & lookups
src/Dango/DangoGenerator.cs, src/Dango/WellKnownTypes.cs
Added WellKnownTypes constants and switched registrar type/member lookups to use those constants; removed an unused using.
Analysis & Codegen (cosmetic)
src/Dango/Analysis/MappingAnalyzer.cs, src/Dango/Analysis/MappingResolver.cs, src/Dango/CodeGeneration/CodeGenerator.cs
Formatting and whitespace adjustments only; no behavioral changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I hopped through lines, a gentle tweak,
Net nine to ten — a brighter peak,
New constants whispered names so neat,
Overrides added, mappings meet,
I nibble whitespace, then I streak.

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 21.43% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Update to dotnet10' directly summarizes the main change: upgrading the project from .NET 9.0 to .NET 10.0 across workflows, project files, and dependencies.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
examples/Example.Core/Example.Core/Example.Core.csproj (1)

4-18: Review verified: net10.0 targeting and local analyzer references are intentional.

The CI workflow (dotnet-ci.yml) is aligned with net10.0, and this version is consistently applied across test and example projects. However, note that the release-please.yml workflow uses dotnet-version: 8.0.x, which differs from the project's target framework. Confirm if this version difference in the release workflow is intentional or needs alignment.

@danielpindur danielpindur merged commit e20713e into master Jan 19, 2026
4 checks passed
@danielpindur danielpindur deleted the dotnet10_update branch January 19, 2026 18:40
@danielpindur danielpindur restored the dotnet10_update branch January 19, 2026 18:44
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.

1 participant