Skip to content

feat: 'release pull' command implementation#66

Merged
nterbogt merged 6 commits intomainfrom
release-pull
Apr 2, 2026
Merged

feat: 'release pull' command implementation#66
nterbogt merged 6 commits intomainfrom
release-pull

Conversation

@nterbogt
Copy link
Copy Markdown
Contributor

@nterbogt nterbogt commented Apr 1, 2026

release pull VERSION and release pull VERSION --service fpm commands.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new skpr release pull subcommand to pull the packaged container images associated with a release via the Skpr API, using existing Docker/ECR auth plumbing.

Changes:

  • Introduces internal implementation for pulling release images and parsing image references.
  • Adds Cobra CLI wiring for skpr release pull with an optional --service filter.
  • Threads DockerClientId into the release command so the new subcommand can create the correct Docker client.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
internal/command/release/pull/command.go Implements release image pulling logic (API lookup, auth selection, Docker pull, image ref parsing).
cmd/skpr/release/pull/command.go Adds the skpr release pull Cobra command and --service flag.
cmd/skpr/release/command.go Registers the new pull subcommand and updates NewCommand to accept DockerClientId.
cmd/skpr/main.go Passes featureFlags.DockerClient into release.NewCommand(...).

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

nterbogt and others added 4 commits April 2, 2026 10:43
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ent subcommand examples. The doco already has them
@nterbogt nterbogt merged commit 6127bcf into main Apr 2, 2026
2 checks passed
@nterbogt nterbogt deleted the release-pull branch April 2, 2026 02:48
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.

3 participants