Skip to content

chore: bump x/v0.0.23-do.not.use#477

Merged
joe0BAB merged 1 commit intomainfrom
x/bump
Feb 27, 2026
Merged

chore: bump x/v0.0.23-do.not.use#477
joe0BAB merged 1 commit intomainfrom
x/bump

Conversation

@joe0BAB
Copy link
Collaborator

@joe0BAB joe0BAB commented Feb 27, 2026

No description provided.

Signed-off-by: Johannes Großmann <grossmann.johannes@t-online.de>
Base automatically changed from x/paths to main February 27, 2026 10:09
@joe0BAB joe0BAB marked this pull request as ready for review February 27, 2026 10:32
@joe0BAB joe0BAB merged commit a5345a3 into main Feb 27, 2026
19 of 20 checks passed
@joe0BAB joe0BAB deleted the x/bump branch February 27, 2026 10:33
Copy link

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Review Summary

This PR cleanly bumps the internal x module dependency from v0.0.22-do.not.use to v0.0.23-do.not.use across all modules. The changes are consistent and properly update:

  • All go.mod files referencing the x module
  • The go.sum file in plugins/credentialhelper (the only one that needs updating)
  • The vendor/modules.txt file

All modules use replace directives pointing to local directories (../x or ./x), which means go.sum entries for the x module are not required in the other modules. The PR looks good!

Copy link

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

✅ Review Complete

This dependency version bump looks good! All occurrences of github.com/docker/secrets-engine/x have been consistently updated from v0.0.22-do.not.use to v0.0.23-do.not.use across all affected files:

  • ✓ All 6 go.mod files updated consistently
  • ✓ go.sum updated with correct hash
  • ✓ vendor/modules.txt updated accordingly

No issues found.

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