Skip to content

Drop .NET 6 and add .NET 11 - #891

Merged
304NotModified merged 12 commits into
masterfrom
net11
Nov 10, 2026
Merged

304NotModified merged 12 commits into
masterfrom
net11

Conversation

@304NotModified

@304NotModified 304NotModified commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

Nice, .NET 11 doesn't need the abstractions references :) - see dotnet/docs#52865

  • Update checks

Note:

  • Tests run on Linux for both .NET 10 and .NET 11. Since .NET 10 is LTS, I'd prefer to have this tested.
  • The .NET 8 tests require Microsoft.Extensions.* packages v10.
Image

@pull-request-size pull-request-size Bot added size/M and removed size/S labels Nov 10, 2026
@codecov

codecov Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.97%. Comparing base (55154f4) to head (7432201).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #891      +/-   ##
==========================================
+ Coverage   75.70%   75.97%   +0.27%     
==========================================
  Files          23       23              
  Lines        1457     1457              
  Branches      430      430              
==========================================
+ Hits         1103     1107       +4     
+ Misses        173      169       -4     
  Partials      181      181              

☔ View full report in Codecov by Harness.
📢 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.

@304NotModified

Copy link
Copy Markdown
Member Author
Image

@pull-request-size pull-request-size Bot added size/L and removed size/M labels Nov 10, 2026
@304NotModified 304NotModified changed the title .NET 11 test Drop .NET 6 and add .NET 11 Nov 10, 2026
Comment thread .github/ISSUE_TEMPLATE.md Resolved
Comment thread src/NLog.Extensions.Logging/README.md Resolved
@304NotModified
304NotModified marked this pull request as ready for review November 10, 2026 19:22
@304NotModified 304NotModified linked an issue Nov 10, 2026 that may be closed by this pull request
@304NotModified 304NotModified added this to the 6.3.0 milestone Nov 10, 2026
@sonarqubecloud

Copy link
Copy Markdown

@304NotModified
304NotModified merged commit e1bde4c into master Nov 10, 2026
15 checks passed
@304NotModified
304NotModified deleted the net11 branch November 10, 2026 21:13
Comment thread .github/ISSUE_TEMPLATE.md
**NLog.Web.AspNetCore version**: (e.g. 4.5.6) (or not used)

**Platform**: .Net 4.6.1 / .NET Core 1 / .NET Core 2 / .NET Core 3.1 / .NET 6.0
**Platform**: .NET Framework 4.6.2 / .NET Standard 2.0 or 2.1 / .NET 8, 10 or 11

@snakefoot snakefoot Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do not think .NET Standard is an Application-Platform. Maybe put slashes between .NET 8 / .NET 10 (Maybe leave out NET11 and only keep LTS, like already excluding NET 9)

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add .NET 11 and remove .NET 6

2 participants

⌂ 换网址