Skip to content

Implement convenience command depot login-docker#509

Open
agdphd wants to merge 5 commits into
depot:mainfrom
agdphd:agdphd/configure-docker
Open

Implement convenience command depot login-docker#509
agdphd wants to merge 5 commits into
depot:mainfrom
agdphd:agdphd/configure-docker

Conversation

@agdphd
Copy link
Copy Markdown
Contributor

@agdphd agdphd commented May 13, 2026

Before I start rolling out workflows involving the Depot registry to my org, I'd like to have a nice one-liner for them to make sure they're logged in to Docker. This PR adds a depot login-docker command that takes care of the details.

TESTED: I've confirmed locally that this gets docker pull <org>.registry.depot.dev/[...] working from a logged-out state.


Note

Medium Risk
Adds a new CLI command that logs into Docker registries and persists credentials, which affects local auth/credential storage and could impact users’ Docker config if misused.

Overview
Adds a new depot login-docker command that logs Docker into {org}.registry.depot.dev using the user Depot API token and stores it in the Docker credential store for docker pull.

Registers the command in the root CLI and documents it in README.md, alongside a small docs table formatting fix for projects delete flags.

Reviewed by Cursor Bugbot for commit 1de5c65. Bugbot is set up for automated code reviews on this repo. Configure here.

Comment thread pkg/cmd/docker/login.go Outdated
Comment thread pkg/cmd/docker/login.go
Comment thread pkg/cmd/docker/login.go
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.

Reviewed by Cursor Bugbot for commit 8b2dd49. Configure here.

Comment thread pkg/cmd/docker/login.go Outdated
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.

1 participant