Skip to content

feat: add dotnet 10 support#328

Draft
wmundev wants to merge 4 commits into
twcclegg:mainfrom
wmundev:feature/add-dotnet10
Draft

feat: add dotnet 10 support#328
wmundev wants to merge 4 commits into
twcclegg:mainfrom
wmundev:feature/add-dotnet10

Conversation

@wmundev
Copy link
Copy Markdown
Collaborator

@wmundev wmundev commented May 6, 2026

Changes

  • add dotnet 10 support

@wmundev wmundev self-assigned this May 7, 2026
Comment thread appveyor.yml Outdated
Co-authored-by: Thomas Clegg <t@clegg.dev>
@wmundev wmundev closed this May 13, 2026
# Conflicts:
#	.github/workflows/run_performance_tests_windows.yml
#	csharp/PhoneNumbers.Extensions.Test/PhoneNumbers.Extensions.Test.csproj
@wmundev wmundev reopened this May 13, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.94%. Comparing base (aec0cdb) to head (49da31d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #328   +/-   ##
=======================================
  Coverage   97.94%   97.94%           
=======================================
  Files          40       40           
  Lines       52924    52924           
  Branches     1115     1115           
=======================================
  Hits        51839    51839           
  Misses        851      851           
  Partials      234      234           

☔ 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.

@wmundev wmundev changed the title [DO NOT MERGE YET] feat: add dotnet 10 support feat: add dotnet 10 support May 15, 2026
- name: Run benchmarks (main branch)
if: github.event_name == 'pull_request'
run: dotnet run -c Release --framework net9.0 -- --filter "*"
run: dotnet run -c Release --framework net10.0 -- --filter "*"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Do we want to just merge this PR as is even though the CI is failing? It's just because net10 doesn't exist on main yet, which it will once this is merged. Or can do it as a two step process and leave this as net9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants