-
Notifications
You must be signed in to change notification settings - Fork 1
V9.0.6/service update #14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughAdds 9.0.6 release entries across package release notes, updates CHANGELOG.md with 9.0.6, bumps several NuGet dependencies via Directory.Packages.props, and updates a Docker test image tag in testenvironments.json. Some prior dependency notes are relocated/removed in release notes. No code or API changes. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this 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 package dependencies across the project and documents these changes in release notes for version 9.0.6. The update includes upgrading various Codebelt and Cuemon libraries, test runners, and ASP.NET Core components to their latest compatible versions.
- Updated multiple package versions in the central package management file
- Updated Docker test environment image to a newer version
- Added release documentation for version 9.0.6 across all affected NuGet packages
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| Directory.Packages.props | Updated package versions for Codebelt.Extensions.Xunit, Cuemon libraries, xunit.runner.visualstudio, and Microsoft.AspNetCore.Mvc.NewtonsoftJson |
| testenvironments.json | Updated Docker test runner image version |
| CHANGELOG.md | Added changelog entry for version 9.0.6 service update |
| .nuget/*/PackageReleaseNotes.txt | Added release notes for version 9.0.6 across four NuGet packages |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
There was a problem hiding this 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
🧹 Nitpick comments (6)
testenvironments.json (1)
12-12: Pin Docker image by digest for reproducible CIThe tag
net8.0.413-9.0.304exists and resolves to
sha256:72470f0d6b22c3d6f8683f6174470a786c9add57f55f76fe603e15a1d9d14594.
To avoid surprise breakages when tags are re-published, consider pinning by digest:File: testenvironments.json
Lines: 12- "dockerImage": "gimlichael/ubuntu-testrunner:net8.0.413-9.0.304" + "dockerImage": "gimlichael/ubuntu-testrunner@sha256:72470f0d6b22c3d6f8683f6174470a786c9add57f55f76fe603e15a1d9d14594"This change is optional but will ensure fully deterministic CI runs.
.nuget/Codebelt.Extensions.AspNetCore.Mvc.Formatters.Newtonsoft.Json/PackageReleaseNotes.txt (1)
1-6: Normalize release notes whitespaceConfirmed there’s no UTF-8 BOM in this or any other release-notes file, but this file still contains non-breaking-space characters on otherwise empty lines. These should be plain blank lines to avoid diff noise and tooling quirks.
• File:
.nuget/Codebelt.Extensions.AspNetCore.Mvc.Formatters.Newtonsoft.Json/PackageReleaseNotes.txt
Lines with NBSP-only content: 3, 6, 9, 12, 15, 18, 21, 24, 27, 30, 33, 36, 39, 43, 46, 49, 52, 55, 58, 61, 64, 67, 70, 73, 78, 81, 84, 87, 90, 95, 98Apply this diff (and similarly across other
.nuget/**/PackageReleaseNotes.txtfiles) to replace NBSP-only lines with true blank lines:- +You can batch-clean all release notes with:
find .nuget -name PackageReleaseNotes.txt -exec sed -i 's/\xC2\xA0//g' {} +.nuget/Codebelt.Extensions.Newtonsoft.Json/PackageReleaseNotes.txt (1)
1-6: Normalize encoding: drop BOM and NBSP linesSame nit as other release notes: remove BOM on the first line and replace NBSP-only lines with true empty lines.
-Version 9.0.6 +Version 9.0.6 - + # ALM - CHANGED Dependencies have been upgraded to the latest compatible versions for all supported target frameworks (TFMs) - +.nuget/Codebelt.Extensions.Newtonsoft.Json.App/PackageReleaseNotes.txt (1)
1-6: Normalize encoding artifacts (BOM and NBSP)Remove the BOM at the start and use plain empty lines instead of NBSP to keep diffs/tooling clean.
-Version 9.0.6 +Version 9.0.6 - + # ALM - CHANGED Dependencies have been upgraded to the latest compatible versions for all supported target frameworks (TFMs) - +.nuget/Codebelt.Extensions.AspNetCore.Newtonsoft.Json/PackageReleaseNotes.txt (2)
1-1: Remove stray BOM (U+FEFF) at file startThe leading invisible BOM character before “Version 9.0.6” can cause tooling noise in diffs and inconsistent rendering. Save the file as UTF‑8 without BOM.
3-6: Normalize blank lines (NBSP characters) to plain empty linesThere are non-breaking spaces used as “blank” lines. Prefer true empty lines to avoid odd rendering in some editors/diff tools.
Apply this diff:
- + # ALM - CHANGED Dependencies have been upgraded to the latest compatible versions for all supported target frameworks (TFMs) - +
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (7)
.nuget/Codebelt.Extensions.AspNetCore.Mvc.Formatters.Newtonsoft.Json/PackageReleaseNotes.txt(1 hunks).nuget/Codebelt.Extensions.AspNetCore.Newtonsoft.Json/PackageReleaseNotes.txt(1 hunks).nuget/Codebelt.Extensions.Newtonsoft.Json.App/PackageReleaseNotes.txt(1 hunks).nuget/Codebelt.Extensions.Newtonsoft.Json/PackageReleaseNotes.txt(1 hunks)CHANGELOG.md(1 hunks)Directory.Packages.props(2 hunks)testenvironments.json(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
- GitHub Check: call-test (windows-2022, Debug) / 🧪 Test
- GitHub Check: call-test (windows-2022, Release) / 🧪 Test
- GitHub Check: call-pack (Debug) / 📦 Pack
- GitHub Check: call-pack (Release) / 📦 Pack
- GitHub Check: call-test (ubuntu-24.04, Release) / 🧪 Test
- GitHub Check: call-test (ubuntu-24.04, Debug) / 🧪 Test
🔇 Additional comments (6)
CHANGELOG.md (1)
10-13: LGTM: Clear 9.0.6 entry aligned with PR scopeThe new entry is concise and consistent with prior service updates. Date matches the PR metadata.
.nuget/Codebelt.Extensions.AspNetCore.Newtonsoft.Json/PackageReleaseNotes.txt (2)
1-2: 9.0.6 release entry and availability look goodClear, consistent with the PR scope (service update + dependency bumps). No issues in the headline/availability lines.
7-7: Release notes consistency confirmedAll four
.nuget/**/PackageReleaseNotes.txtfiles include:
Version 9.0.5on line 7Availability: .NET 9 and .NET 8in each header blockNo further edits are needed—each package’s release notes are aligned.
Directory.Packages.props (3)
24-24: xunit.runner.visualstudio -> 3.1.4 looks fineCompatible with xunit 2.9.3 in practice. No concern here.
6-15: Directory.Packages.props: Package bumps verifiedAll bumped packages match the latest stable releases with no API-surface changes:
- Codebelt.Extensions.Xunit / .App → 10.0.5
- Cuemon.* → 9.0.8
- xunit.runner.visualstudio → 3.1.4
Approved ✅
Optional refactor to reduce future maintenance: centralize the Cuemon version by adding at the top of Directory.Packages.props
<PropertyGroup> <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally> <CuemonVersion>9.0.8</CuemonVersion> </PropertyGroup>and updating each Cuemon entry to:
- <PackageVersion Include="Cuemon.Core" Version="9.0.8" /> + <PackageVersion Include="Cuemon.Core" Version="$(CuemonVersion)" />(Optional housekeeping: audit any legacy pins such as System.Net.Http 4.3.4 in a follow-up.)
27-27: TFM-conditional NewtonsoftJson versions are up-to-dateVerified against NuGet.org:
- net9*: 9.0.8 (latest 9.x)
- net8*: 8.0.19 (latest 8.x)
No further changes required.



This pull request is a service update focused on upgrading package dependencies across the project. It updates dependency versions for several key libraries and test tools, and documents these changes in the release notes and changelog for relevant NuGet packages.
Dependency upgrades:
Codebelt.Extensions.Xunit,Codebelt.Extensions.Xunit.App, and multipleCuemon.*libraries to their latest compatible versions inDirectory.Packages.props. Also updatedxunit.runner.visualstudioandMicrosoft.AspNetCore.Mvc.NewtonsoftJsonfor .NET 8 and .NET 9 target frameworks. [1] [2]gimlichael/ubuntu-testrunner:net8.0.413-9.0.304intestenvironments.json.Release documentation:
PackageReleaseNotes.txtfiles forCodebelt.Extensions.AspNetCore.Mvc.Formatters.Newtonsoft.Json,Codebelt.Extensions.AspNetCore.Newtonsoft.Json,Codebelt.Extensions.Newtonsoft.Json.App, andCodebelt.Extensions.Newtonsoft.Json, noting the dependency upgrades and supported frameworks. [1] [2] [3] [4]CHANGELOG.md, describing this as a service update focused on package dependencies.Summary by CodeRabbit
Documentation
Chores
Notes