-
Notifications
You must be signed in to change notification settings - Fork 1
V10.0.5/service update #42
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 10.0.5 release notes across NuGet packages, updates CHANGELOG with a 2025-08-16 entry, bumps central dependency versions in Directory.Packages.props (.NET 9/8 packages, Cuemon, xunit runner), and updates the Docker test image tag in testenvironments.json. No code or public API changes. Changes
Sequence Diagram(s)Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
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 pull request updates the project to version 10.0.5, focusing on upgrading package dependencies to their latest compatible versions across all supported target frameworks and updating the corresponding documentation.
- Upgraded Cuemon packages from 9.0.7 to 9.0.8
- Updated Microsoft Extensions packages for .NET 9 from 9.0.7 to 9.0.8
- Updated test tooling and Docker image versions to latest patch releases
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
File | Description |
---|---|
Directory.Packages.props | Updated dependency versions for Cuemon packages, Microsoft Extensions packages, and test tooling |
testenvironments.json | Updated Docker test runner image to latest .NET patch versions |
CHANGELOG.md | Added changelog entry for version 10.0.5 service update |
.nuget/*/PackageReleaseNotes.txt | Updated package release notes to version 10.0.5 with dependency upgrade information |
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: 1
🧹 Nitpick comments (5)
.nuget/Codebelt.Extensions.Xunit.Hosting/PackageReleaseNotes.txt (1)
1-6
: Remove UTF-8 BOM and NBSP-only spacer line for consistencyThere’s a BOM before “Version” (invisible “”) and a non-breaking space line used as a separator. Prefer UTF-8 without BOM and plain blank lines to avoid odd diffs or tooling hiccups.
Apply this minimal cleanup:
-Version 10.0.5 +Version 10.0.5 @@ - +.nuget/Codebelt.Extensions.Xunit/PackageReleaseNotes.txt (1)
1-6
: Normalize encoding and spacer lineSame as other release notes: remove the UTF-8 BOM and replace the NBSP-only line with a regular blank line.
-Version 10.0.5 +Version 10.0.5 @@ - +.nuget/Codebelt.Extensions.Xunit.App/PackageReleaseNotes.txt (1)
1-6
: Strip BOM and NBSP-only lineFor consistent text files and cleaner diffs, remove the BOM and use a regular empty line as separator.
-Version 10.0.5 +Version 10.0.5 @@ - +.nuget/Codebelt.Extensions.Xunit.Hosting.AspNetCore/PackageReleaseNotes.txt (1)
1-6
: Nit: remove BOM and normalize whitespace (NBSP) on the blank line.Line 1 appears to include a UTF-8 BOM, and line 6 is a non-breaking space. These can render oddly in some consumers (NuGet/Markdown viewers). Prefer UTF-8 without BOM and true empty lines.
Apply this diff to replace the NBSP-only line with an actual empty line; also consider saving the file without BOM.
- +Directory.Packages.props (1)
40-40
: net8 TestHost pinned to 8.0.19 — good; optional: align other net8 Extensions packages to latest 8.0.xThe TestHost update is on the latest 8.0.x stable release. Other Microsoft.Extensions.* packages currently sit at 8.0.0/8.0.1. If you’d like to ensure all Extensions packages use the newest 8.0.x patch, run this script:
#!/usr/bin/env bash set -euo pipefail pkgs=( "Microsoft.Extensions.Hosting" "Microsoft.Extensions.Configuration" "Microsoft.Extensions.Configuration.EnvironmentVariables" "Microsoft.Extensions.Configuration.FileExtensions" "Microsoft.Extensions.Configuration.Json" ) for name in "${pkgs[@]}"; do echo "→ $name" lid=${name,,} latest=$(curl -sS "https://api.nuget.org/v3-flatcontainer/${lid}/index.json" \ | jq -r ' .versions | map(select(startswith("8.") and (test("-")|not))) | sort_by(split(".")|map(tonumber)) | last // "N/A" ') echo " latest stable 8.x = $latest" doneReview the output and update any out-of-date package pins in Directory.Packages.props as desired.
📜 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.Xunit.App/PackageReleaseNotes.txt
(1 hunks).nuget/Codebelt.Extensions.Xunit.Hosting.AspNetCore/PackageReleaseNotes.txt
(1 hunks).nuget/Codebelt.Extensions.Xunit.Hosting/PackageReleaseNotes.txt
(1 hunks).nuget/Codebelt.Extensions.Xunit/PackageReleaseNotes.txt
(1 hunks)CHANGELOG.md
(1 hunks)Directory.Packages.props
(2 hunks)testenvironments.json
(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: gimlichael
PR: codebeltnet/xunit#17
File: src/Codebelt.Extensions.Xunit.Hosting/Codebelt.Extensions.Xunit.Hosting.csproj:22-26
Timestamp: 2024-10-09T14:10:40.721Z
Learning: In the `src/Codebelt.Extensions.Xunit.Hosting/Codebelt.Extensions.Xunit.Hosting.csproj` file, only certain .NET 8 packages are available in version 8.0.1, while others remain at 8.0.0, so package versions may differ due to availability.
📚 Learning: 2024-10-09T14:10:40.721Z
Learnt from: gimlichael
PR: codebeltnet/xunit#17
File: src/Codebelt.Extensions.Xunit.Hosting/Codebelt.Extensions.Xunit.Hosting.csproj:22-26
Timestamp: 2024-10-09T14:10:40.721Z
Learning: In the `src/Codebelt.Extensions.Xunit.Hosting/Codebelt.Extensions.Xunit.Hosting.csproj` file, only certain .NET 8 packages are available in version 8.0.1, while others remain at 8.0.0, so package versions may differ due to availability.
Applied to files:
CHANGELOG.md
Directory.Packages.props
.nuget/Codebelt.Extensions.Xunit.Hosting/PackageReleaseNotes.txt
.nuget/Codebelt.Extensions.Xunit/PackageReleaseNotes.txt
.nuget/Codebelt.Extensions.Xunit.App/PackageReleaseNotes.txt
.nuget/Codebelt.Extensions.Xunit.Hosting.AspNetCore/PackageReleaseNotes.txt
⏰ 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). (1)
- GitHub Check: call-build (Debug) / 🛠️ Build
🔇 Additional comments (8)
CHANGELOG.md (1)
10-13
: Changelog entry for 10.0.5 looks goodAccurately reflects a service update focused on dependencies; consistent with surrounding entries.
.nuget/Codebelt.Extensions.Xunit.Hosting/PackageReleaseNotes.txt (1)
1-6
: 10.0.5 release notes read well and align with PR scopeClear availability and ALM note; consistent with other packages in this PR.
.nuget/Codebelt.Extensions.Xunit/PackageReleaseNotes.txt (1)
1-6
: 10.0.5 ALM entry is clear and matches the dependency-bump themeAvailability and wording match the rest of the repo updates.
.nuget/Codebelt.Extensions.Xunit.App/PackageReleaseNotes.txt (1)
1-6
: Release note for 10.0.5 is concise and consistentMatches the dependency-upgrade nature of this service release.
.nuget/Codebelt.Extensions.Xunit.Hosting.AspNetCore/PackageReleaseNotes.txt (1)
1-6
: 10.0.5 release entry looks good and matches PR scope.Clear ALM note; availability matches supported TFMs; no API claims. LGTM.
Directory.Packages.props (3)
23-23
: xunit.runner.visualstudio → 3.1.4 — OK.Adapter update is fine alongside xunit 2.9.3. If you run tests in Azure DevOps/VS Test Platform, ensure no adapter behavior changes affect discovery.
You can validate by running your CI test stage and checking for any discovery warnings. If desired, I can script a repository scan for vstest task usage.
27-32
: net9 Microsoft.Extensions + TestHost → 9.0.8 — consistent set.The patch level is aligned across the net9 stack; this should reduce runtime-binding noise.
Reuse the NuGet verification script above to confirm presence of 9.0.8 for each package.
7-10
: Cuemon & BCL pins at 9.0.8 verified on NuGet
All four packages in Directory.Packages.props (Lines 7–10) exist at version 9.0.8 on nuget.org—no restore issues expected.
"name": "Docker-Ubuntu", | ||
"type": "docker", | ||
"dockerImage": "gimlichael/ubuntu-testrunner:net8.0.412-9.0.302" | ||
"dockerImage": "gimlichael/ubuntu-testrunner:net8.0.413-9.0.304" |
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.
💡 Verification agent
🧩 Analysis chain
Docker test runner tag bump looks correct; verify tag exists
The image pin moves to net8.0.413-9.0.304 which aligns with latest SDK patch updates. Recommend verifying the tag exists on Docker Hub to avoid CI failures.
Run this script to confirm the tag is available:
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
IMAGE="gimlichael/ubuntu-testrunner"
TAG="net8.0.413-9.0.304"
resp="$(curl -s "https://hub.docker.com/v2/repositories/${IMAGE}/tags?page_size=1&name=${TAG}")"
name="$(printf '%s' "$resp" | jq -r '.results[0].name // empty')"
if [[ "$name" == "$TAG" ]]; then
echo "OK: ${IMAGE}:${TAG} exists on Docker Hub."
else
echo "ERROR: ${IMAGE}:${TAG} not found on Docker Hub."
echo "Response snippet:"
printf '%s' "$resp" | jq '.results'
exit 1
fi
Length of output: 6097
Docker test runner tag doesn’t exist on Docker Hub—please correct
The tag "net8.0.413-9.0.304" referenced in testenvironments.json is not found; Docker Hub only returns "mono-net8.0.413-9.0.304". Update the tag (or publish the intended tag) to avoid CI failures.
- File: testenvironments.json, line 12
Change
"dockerImage": "gimlichael/ubuntu-testrunner:net8.0.413-9.0.304"
to
"dockerImage": "gimlichael/ubuntu-testrunner:mono-net8.0.413-9.0.304"
🤖 Prompt for AI Agents
In testenvironments.json around line 12, the dockerImage tag
"gimlichael/ubuntu-testrunner:net8.0.413-9.0.304" does not exist on Docker Hub;
update the value to the existing tag
"gimlichael/ubuntu-testrunner:mono-net8.0.413-9.0.304" (or publish the intended
tag), ensuring the JSON string is replaced exactly and the file remains valid
JSON so CI uses the correct image.
|
This pull request is a service update focused on upgrading dependencies to their latest compatible versions across all supported target frameworks. It also updates release notes and the changelog to reflect these changes.
Dependency Upgrades:
Directory.Packages.props
, includingCuemon.Core
,Cuemon.Extensions.AspNetCore
,Cuemon.Extensions.IO
,Microsoft.Bcl.AsyncInterfaces
, and several Microsoft.Extensions packages for .NET 9 and .NET 8. Also updatedxunit.runner.visualstudio
andMicrosoft.AspNetCore.TestHost
. [1] [2]Documentation and Release Notes:
PackageReleaseNotes.txt
files to version 10.0.5, noting dependency upgrades and clarifying supported target frameworks. [1] [2] [3] [4]CHANGELOG.md
, describing this as a service update focused on package dependencies.Test Environment:
testenvironments.json
to use the latest patch versions for .NET 8 and .NET 9.Summary by CodeRabbit
New Release
Chores
Documentation