Skip to content

general: bump internal dependencies#54

Merged
piotrpio merged 1 commit intomainfrom
bump-internal-dependencies
Apr 1, 2026
Merged

general: bump internal dependencies#54
piotrpio merged 1 commit intomainfrom
bump-internal-dependencies

Conversation

@piotrpio
Copy link
Copy Markdown
Collaborator

Signed-off-by: Piotr Piotrowski piotr@synadia.com

Signed-off-by: Piotr Piotrowski <piotr@synadia.com>
@synadia-claude-reviewer
Copy link
Copy Markdown

synadia-claude-reviewer Bot commented Mar 26, 2026

Claude finished @piotrpio's task in 56s —— View job


PR Review

  • Gather context
  • Examine PR diff
  • Review dependency changes
  • Post final review

This is a clean internal dependency bump across counters and pcgroups modules:

Module Dependency Old New
counters jetstreamext v0.2.0 v0.2.1
counters natsext v0.1.1 v0.1.2
pcgroups natscontext v0.1.1 v0.1.2

All go.sum hashes are updated consistently. One minor observation: in counters/test/go.sum, the new entries for jetstreamext and natsext only include the module zip hash (h1:) and omit the /go.mod h1: lines that were present before. This is expected — counters/test/go.mod uses a replace directive for the local counters module, so Go doesn't require the go.mod checksums for these transitively-resolved indirect dependencies.

No issues found. LGTM.

@piotrpio piotrpio requested a review from Jarema April 1, 2026 16:07
Copy link
Copy Markdown
Collaborator

@Jarema Jarema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@piotrpio piotrpio merged commit 237e444 into main Apr 1, 2026
10 checks passed
@piotrpio piotrpio deleted the bump-internal-dependencies branch April 1, 2026 16:13
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