Conversation
📝 WalkthroughWalkthroughThis pull request updates version 1.2.2 release information, bumps several dependencies, updates copyright years to 2025-2026, restructures the CI/CD pipeline to add architecture-specific testing matrices, and upgrades Docker and test environment images. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@Directory.Packages.props`:
- Around line 9-14: The two PackageVersion entries for coverlet.collector and
coverlet.msbuild in Directory.Packages.props reference a non-existent version
(8.0.0); update those PackageVersion Include="coverlet.collector" and
PackageVersion Include="coverlet.msbuild" entries to a valid released version
(e.g., 6.0.4) or the intended stable release, then restore/build to verify
resolution.
🧹 Nitpick comments (1)
.docfx/Dockerfile.docfx (1)
1-1: Consider adding a non-rootUSERdirective.Trivy flags that this Dockerfile runs as root. While this is a docs-serving container and not a production API, running nginx as a non-root user is a security best practice that reduces the blast radius of container escapes.
This is a pre-existing pattern, so it can be addressed separately.
🔒 Proposed fix to run as non-root
FROM nginx:${NGINX_VERSION} AS final WORKDIR /usr/share/nginx/html COPY --from=build /build/docfx/wwwroot /usr/share/nginx/html + +USER nginx ENTRYPOINT ["nginx", "-g", "daemon off;"]Note: You may need to adjust file permissions or nginx config to ensure the
nginxuser can bind to port 80 (or switch to a higher port like 8080).
| <PackageVersion Include="Codebelt.Extensions.Xunit.App" Version="11.0.5" /> | ||
| <PackageVersion Include="Cuemon.Core" Version="10.2.1" /> | ||
| <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.0.1" /> | ||
| <PackageVersion Include="MinVer" Version="7.0.0" /> | ||
| <PackageVersion Include="coverlet.collector" Version="6.0.4" /> | ||
| <PackageVersion Include="coverlet.msbuild" Version="6.0.4" /> | ||
| <PackageVersion Include="coverlet.collector" Version="8.0.0" /> | ||
| <PackageVersion Include="coverlet.msbuild" Version="8.0.0" /> |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
coverlet 8.0.0 breaking changes from 6.x migration guide
💡 Result:
As of Feb 2026, upstream coverlet has not released any 8.0.0 package/version—GitHub “Releases” shows the latest as v6.0.4 (Jan 19, 2025). [1] So there’s no official “6.x → 8.0.0” migration guide.
What you can migrate (6.x notes / breaking changes)
-
If you’re coming from <6.0.0 to 6.x:
InstrumentModulesWithoutLocalSourceswas removed. Use the newExcludeAssembliesWithoutSourcesinstead (e.g., set it toNoneto get the old “instrument even without local sources” behavior). [2]- The default instrumentation heuristic changed: previously, any missing source file could cause an assembly to be skipped; now, only assemblies where no source files are found are excluded. You can approximate the old behavior with
ExcludeAssembliesWithoutSources=MissingAny(or use explicit assembly exclude filters). [2]
-
If you’re already on 6.x: there are no additional breaking changes documented after 6.0.0—6.0.1–6.0.4 are fixes/improvements in the changelog. [2]
Upcoming breaking change (not released yet)
- The changelog “Unreleased” section notes a planned breaking change: minimum required .NET SDK/runtime will be raised to .NET 8 LTS. [2]
If you meant a different package named “coverlet 8.0.0” (fork/third-party), tell me the exact NuGet package ID (e.g., coverlet.collector, coverlet.msbuild, etc.) and where you saw 8.0.0, and I’ll map the migration to that artifact.
Fix invalid coverlet package version: 8.0.0 does not exist in official releases.
The Directory.Packages.props file specifies coverlet.collector and coverlet.msbuild at version 8.0.0, but this version has not been released. As of February 2026, the latest official coverlet release is 6.0.4 (Jan 19, 2025). Builds will fail when attempting to resolve these non-existent package versions. Update to the latest stable version (6.0.4) or verify if a specific version was intended.
🤖 Prompt for AI Agents
In `@Directory.Packages.props` around lines 9 - 14, The two PackageVersion entries
for coverlet.collector and coverlet.msbuild in Directory.Packages.props
reference a non-existent version (8.0.0); update those PackageVersion
Include="coverlet.collector" and PackageVersion Include="coverlet.msbuild"
entries to a valid released version (e.g., 6.0.4) or the intended stable
release, then restore/build to verify resolution.
There was a problem hiding this comment.
Pull request overview
Service update for v1.2.2 focused on dependency upgrades and expanding CI coverage across OS/architecture, with accompanying release-note and copyright refreshes.
Changes:
- Bumped central NuGet dependency versions (notably Codebelt.Extensions.Xunit.App, Cuemon.Core, coverlet).
- Updated CI workflow to build/test across Linux+Windows and X64+ARM64, and adjusted downstream job dependencies.
- Added v1.2.2 release notes / changelog entry and refreshed copyright years; updated Docker-based tooling images/versions.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
Directory.Packages.props |
Central package version upgrades for the release. |
.github/workflows/ci-pipeline.yml |
Split test jobs by OS and added X64/ARM64 matrix support; updated downstream dependencies. |
testenvironments.json |
Updated docker test runner image reference. |
.docfx/Dockerfile.docfx |
Updated nginx version used for docfx container build. |
.docfx/docfx.json |
Updated generated documentation footer copyright years. |
CHANGELOG.md |
Added v1.2.2 entry with service update note. |
.nuget/Codebelt.Extensions.BenchmarkDotNet/PackageReleaseNotes.txt |
Added v1.2.2 release notes. |
.nuget/Codebelt.Extensions.BenchmarkDotNet.Console/PackageReleaseNotes.txt |
Added v1.2.2 release notes. |
Directory.Build.props |
Updated assembly/package copyright metadata. |
LICENSE |
Updated copyright year range. |
| with: | ||
| configuration: ${{ matrix.configuration }} | ||
| version: ${{ needs.build.outputs.version }} | ||
| download-build-artifact-pattern: build-${{ matrix.configuration }}-X64 |
There was a problem hiding this comment.
pack now downloads only the X64 build artifact (download-build-artifact-pattern: ...-X64), but it still needs: [build] where build runs a full X64+ARM64 matrix. This makes pack wait for ARM64 builds it never consumes, increasing pipeline duration. Consider restructuring so pack only depends on the specific build artifact it needs (e.g., a dedicated X64 build job used for packing/versioning), while keeping ARM64 builds for test validation.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7 +/- ##
=======================================
Coverage 66.95% 66.95%
=======================================
Files 6 6
Lines 345 345
Branches 42 42
=======================================
Hits 231 231
Misses 114 114 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
1 participant
This pull request delivers a service update focused on upgrading package dependencies and expanding CI pipeline support for multiple architectures and platforms. It also updates copyright and documentation to reflect the new release.
Dependency and Release Updates:
Codebelt.Extensions.Xunit.App,Cuemon.Core, andcoverletpackages.PackageReleaseNotes.txtfiles and updated theCHANGELOG.mdto document the new release as a service update focused on dependencies. [1] [2] [3]CI/CD Pipeline Improvements:
test_linuxandtest_windowsjobs, and updated artifact handling accordingly. [1] [2]sonarcloud,codecov,codeql,deploy) to depend on the new architecture/platform-specific test jobs.Documentation and Legal Updates:
LICENSE,Directory.Build.props, and documentation/config files to include 2026. [1] [2] [3]Summary by CodeRabbit
Chores
Documentation