镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Target .NET 11 - #842

Merged
erwan-joly merged 2 commits into
masterfrom
claude/project-thread-451h4z
Nov 11, 2026
Merged

erwan-joly merged 2 commits into
masterfrom
claude/project-thread-451h4z

Conversation

@erwan-joly

@erwan-joly erwan-joly commented Nov 11, 2026 •

Copy link
Copy Markdown
Collaborator

Requested by Erwan · project thread

Moves the bot and its tests from net10.0 to net11.0, the CI to the 11.0 SDK, and the Microsoft.Extensions/AspNetCore hosting packages to 11.0.0. Builds locally on SDK 11.0.100.


Generated by Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the application and its automated checks to target .NET 11.
    • Updated the Windows service hosting and dependency injection components to version 11.0.0.
    • Updated the build workflow to use .NET 11; restore, build, and test commands remain unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014gSmkX8c2PxcxJdoLXzPqf
@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Explain Change →

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 066395a and 25c429d.

Files selected for processing: 1

  • src/NosCoreBot/dockerfile

No actionable comments were generated in the recent review.

Walkthrough

The application and test project now target .NET 11. The CI workflow requests .NET 11, and the Windows Services hosting and dependency-injection packages are updated to version 11.0.0.

Changes

.NET 11 Upgrade

Layer / File(s) Summary
Update .NET target and build setup
.github/workflows/dotnet.yml, src/NosCoreBot/NosCoreBot.csproj, tests/NosCoreBot.Tests/NosCoreBot.Tests.csproj
The application and test project target .NET 11. The CI workflow requests .NET 11. The Windows Services hosting and dependency-injection packages move from version 10.0.14 to 11.0.0.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Possibly related PRs

  • NosCoreIO/NosCoreBot#836: Updates the same Microsoft hosting packages to version 10.0.14, which this PR advances to 11.0.0.
Merge Risk: Minimal · previous review 06639

The upgrade has no identified merge-blocking issue.

Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: upgrading the project to target .NET 11.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between a32937e and 066395a.

Files selected for processing: 3

  • .github/workflows/dotnet.yml
  • src/NosCoreBot/NosCoreBot.csproj
  • tests/NosCoreBot.Tests/NosCoreBot.Tests.csproj

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014gSmkX8c2PxcxJdoLXzPqf
@erwan-joly
erwan-joly merged commit d17bda9 into master Nov 11, 2026
3 checks passed
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