Bump the nuget-production-dependencies group with 3 updates#7
Conversation
Bumps Grpc.AspNetCore from 2.67.0 to 2.76.0 Bumps Grpc.AspNetCore.HealthChecks from 2.67.0 to 2.76.0 Bumps Grpc.Tools from 2.68.1 to 2.80.0 --- updated-dependencies: - dependency-name: Grpc.Tools dependency-version: 2.80.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-production-dependencies - dependency-name: Grpc.AspNetCore dependency-version: 2.76.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-production-dependencies - dependency-name: Grpc.AspNetCore.HealthChecks dependency-version: 2.76.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-production-dependencies - dependency-name: Grpc.AspNetCore dependency-version: 2.76.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-production-dependencies - dependency-name: Grpc.AspNetCore.HealthChecks dependency-version: 2.76.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-production-dependencies - dependency-name: Grpc.AspNetCore dependency-version: 2.76.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-production-dependencies - dependency-name: Grpc.AspNetCore.HealthChecks dependency-version: 2.76.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: nuget-production-dependencies ... Signed-off-by: dependabot[bot] <support@github.com>
| @@ -16,8 +16,8 @@ | |||
|
|
|||
| <ItemGroup> | |||
| <!-- Keeping Grpc.AspNetCore* to 2.67 due to https://github.com/grpc/grpc/issues/38538 --> | |||
There was a problem hiding this comment.
WARNING: Comment about pinning to 2.67.0 due to issue #38538 is now outdated/stale since the version was updated to 2.76.0. Consider updating or removing this comment to avoid confusion.
| <!-- Keeping Grpc.AspNetCore* to 2.67 due to https://github.com/grpc/grpc/issues/38538 --> | ||
| <PackageReference Include="Grpc.AspNetCore" Version="2.67.0" /> | ||
| <PackageReference Include="Grpc.AspNetCore.HealthChecks" Version="2.67.0" /> | ||
| <PackageReference Include="Grpc.AspNetCore" Version="2.76.0" /> |
There was a problem hiding this comment.
WARNING: Large version jump from 2.67.0 to 2.76.0 (9 minor versions). Ensure compatibility with the existing codebase and that issue #38538 is resolved in this version.
| <PackageReference Include="Grpc.AspNetCore" Version="2.67.0" /> | ||
| <PackageReference Include="Grpc.AspNetCore.HealthChecks" Version="2.67.0" /> | ||
| <PackageReference Include="Grpc.AspNetCore" Version="2.76.0" /> | ||
| <PackageReference Include="Grpc.AspNetCore.HealthChecks" Version="2.76.0" /> |
There was a problem hiding this comment.
WARNING: Grpc.AspNetCore.HealthChecks also updated to 2.76.0 - ensure compatibility with Grpc.AspNetCore at the same version.
| <PackageReference Include="EFCore.NamingConventions" Version="10.0.1" /> | ||
| <PackageReference Include="Google.Protobuf" Version="3.34.1" /> | ||
| <PackageReference Include="Grpc.Tools" Version="2.68.1"> | ||
| <PackageReference Include="Grpc.Tools" Version="2.80.0"> |
There was a problem hiding this comment.
SUGGESTION: Consider aligning Grpc.Tools version (2.80.0) with Grpc.AspNetCore version (2.76.0) used in cart project to avoid potential tooling/runtime compatibility issues. Mixing minor versions may cause unexpected behavior.
Code Review SummaryStatus: 4 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
SUGGESTION
Other Observations (not in diff)Issues found in unchanged code that cannot receive inline comments:
Files Reviewed (4 files)
Reviewed by ling-2.6-1t-20260423:free · 1,027,905 tokens |
There was a problem hiding this comment.
1 issue found across 2 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="src/cart/src/cart.csproj">
<violation number="1" location="src/cart/src/cart.csproj:19">
P3: These version bumps conflict with the adjacent comment that says the project is intentionally pinned to 2.67. Update or remove the comment to keep dependency intent accurate.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| <!-- Keeping Grpc.AspNetCore* to 2.67 due to https://github.com/grpc/grpc/issues/38538 --> | ||
| <PackageReference Include="Grpc.AspNetCore" Version="2.67.0" /> | ||
| <PackageReference Include="Grpc.AspNetCore.HealthChecks" Version="2.67.0" /> | ||
| <PackageReference Include="Grpc.AspNetCore" Version="2.76.0" /> |
There was a problem hiding this comment.
P3: These version bumps conflict with the adjacent comment that says the project is intentionally pinned to 2.67. Update or remove the comment to keep dependency intent accurate.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At src/cart/src/cart.csproj, line 19:
<comment>These version bumps conflict with the adjacent comment that says the project is intentionally pinned to 2.67. Update or remove the comment to keep dependency intent accurate.</comment>
<file context>
@@ -16,8 +16,8 @@
<!-- Keeping Grpc.AspNetCore* to 2.67 due to https://github.com/grpc/grpc/issues/38538 -->
- <PackageReference Include="Grpc.AspNetCore" Version="2.67.0" />
- <PackageReference Include="Grpc.AspNetCore.HealthChecks" Version="2.67.0" />
+ <PackageReference Include="Grpc.AspNetCore" Version="2.76.0" />
+ <PackageReference Include="Grpc.AspNetCore.HealthChecks" Version="2.76.0" />
<PackageReference Include="OpenTelemetry.Exporter.OpenTelemetryProtocol" Version="1.15.3" />
</file context>
Updated Grpc.AspNetCore from 2.67.0 to 2.76.0.
Release notes
Sourced from Grpc.AspNetCore's releases.
2.76.0
What's Changed
New Contributors
Full Changelog: grpc/grpc-dotnet@v2.71.0...v2.76.0
2.76.0-pre1
What's Changed
New Contributors
Full Changelog: grpc/grpc-dotnet@v2.71.0...v2.76.0-pre1
2.71.0
What's Changed
Full Changelog: grpc/grpc-dotnet@v2.70.0...v2.71.0
2.71.0-pre1
What's Changed
Full Changelog: grpc/grpc-dotnet@v2.70.0...v2.71.0-pre1
2.70.0
What's Changed
await usingforpackageVersionStreamto ensure proper disposal of async resources by @dexcompiler in Refactor: Useawait usingforpackageVersionStreamto ensure proper disposal of async resources grpc/grpc-dotnet#2521New Contributors
await usingforpackageVersionStreamto ensure proper disposal of async resources grpc/grpc-dotnet#2521Full Changelog: grpc/grpc-dotnet@v2.67.0...v2.70.0
Commits viewable in compare view.
Updated Grpc.AspNetCore.HealthChecks from 2.67.0 to 2.76.0.
Release notes
Sourced from Grpc.AspNetCore.HealthChecks's releases.
2.76.0
What's Changed
New Contributors
Full Changelog: grpc/grpc-dotnet@v2.71.0...v2.76.0
2.76.0-pre1
What's Changed
New Contributors
Full Changelog: grpc/grpc-dotnet@v2.71.0...v2.76.0-pre1
2.71.0
What's Changed
Full Changelog: grpc/grpc-dotnet@v2.70.0...v2.71.0
2.71.0-pre1
What's Changed
Full Changelog: grpc/grpc-dotnet@v2.70.0...v2.71.0-pre1
2.70.0
What's Changed
await usingforpackageVersionStreamto ensure proper disposal of async resources by @dexcompiler in Refactor: Useawait usingforpackageVersionStreamto ensure proper disposal of async resources grpc/grpc-dotnet#2521New Contributors
await usingforpackageVersionStreamto ensure proper disposal of async resources grpc/grpc-dotnet#2521Full Changelog: grpc/grpc-dotnet@v2.67.0...v2.70.0
Commits viewable in compare view.
Updated Grpc.Tools from 2.68.1 to 2.80.0.
Release notes
Sourced from Grpc.Tools's releases.
No release notes found for this version range.
Commits viewable in compare view.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditionsSummary by cubic
Upgrade gRPC dependencies to current stable versions in accounting and cart to pick up fixes, performance improvements, and .NET 10 support. Also updates build-time tooling.
Grpc.AspNetCore: 2.67.0 → 2.76.0 (cart)Grpc.AspNetCore.HealthChecks: 2.67.0 → 2.76.0 (cart)Grpc.Tools: 2.68.1 → 2.80.0 (accounting, build-time only)Written for commit 14c3dfc. Summary will update on new commits.