Skip to content

removing obsolete github.com/docker/docker package#1878

Open
jcchr wants to merge 1 commit into
mainfrom
jchrapko/docker_go_update
Open

removing obsolete github.com/docker/docker package#1878
jcchr wants to merge 1 commit into
mainfrom
jchrapko/docker_go_update

Conversation

@jcchr

@jcchr jcchr commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

📝 Description

Update of the obsolete jchrapko/docker_go_update package.

✨ Type of Change

Select the type of change your PR introduces:

  • 🐞 Bug fix – Non-breaking change which fixes an issue
  • 🚀 New feature – Non-breaking change which adds functionality
  • 🔨 Refactor – Non-breaking change which refactors the code base
  • 💥 Breaking change – Changes that break existing functionality
  • 📚 Documentation update
  • 🔒 Security update
  • 🧪 Tests

🧪 Testing Scenarios

Describe how the changes were tested and how reviewers can test them too:

  • ✅ Tested manually
  • 🤖 Run automated end-to-end tests

✅ Checklist

Before submitting the PR, ensure the following:

  • 🔍 PR title is clear and meaningful
  • ✍️ PR description clearly explains the changes and their reason
  • 📝 I have linked the PR to the corresponding GitHub Issues, if any
  • 💬 I have commented my code, especially in hard-to-understand areas
  • 📄 I have made corresponding changes to the documentation
  • ✅ I have added tests that prove my fix is effective or my feature works

Copilot AI review requested due to automatic review settings June 18, 2026 08:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates Go module dependencies across multiple interactive_ai components to eliminate the deprecated github.com/docker/docker module from the dependency graph (primarily by updating upstream dependencies), aligning with the stated security/maintenance intent.

Changes:

  • Removes github.com/docker/docker from the resolved dependency set (no remaining references found in the repo).
  • Updates several transitive dependencies (notably Docker/Moby-related libraries, OpenTelemetry instrumentation, klauspost/compress, golang.org/x/*, and testcontainers-go) across the affected Go modules.
  • Refreshes go.sum files to match the new module graphs.

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
interactive_ai/services/media/go.mod Drops github.com/docker/docker and updates indirect dependency versions to match the new resolved graph.
interactive_ai/services/media/go.sum Updates checksums to reflect the refreshed dependency graph (removal of github.com/docker/docker, new Moby/containerd entries, version bumps).
interactive_ai/services/inference_gateway/go.mod Updates several indirect dependency versions (klauspost/compress, objx, golang.org/x/*).
interactive_ai/services/inference_gateway/go.sum Refreshes checksums and removes github.com/docker/docker from the resolved set.
interactive_ai/libs/iai_core_go/go.mod Updates testcontainers-go (direct) and aligns indirect dependencies with the new graph (including removal of github.com/docker/docker).
interactive_ai/libs/iai_core_go/go.sum Refreshes checksums to match updated dependency resolution and removed obsolete Docker module.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants