Skip to content

Conversation

@lizrabuya
Copy link
Contributor

@lizrabuya lizrabuya commented Dec 12, 2025

Description

Migrate package push command to Kong.

Changes

  • Moved /pkg/cmd/pkg/* to /cmd/pkg/*

  • Modified the conditional requirements of the push command's to use Kong's framework. Added a flag name to specify file path as parameter instead of using positional argument.

Previous usage:

Usage:
  bk package push registry-slug {path/to/file | --stdin-file-name filename -} [flags]

Current Usage:

Usage: bk package push <registry-slug> [<std-in-arg>] [flags]

Arguments:
  <registry-slug>    The slug of the registry to push the package to
  [<std-in-arg>]     Use '-' as value to pass package via stdin. Required if --stdin-file-name is used.

Flags:
      --file-path=STRING          Path to the package file to push
      --stdin-file-name=STRING    The filename to use when reading the package from stdin
  -w, --web                       Open the pipeline in a web browser. 
  • Modified the test to validate command-line arguments and flags only.

Testing

  • Tests have run locally (with go test ./...)
  • Code is formatted (with go fmt ./...)

Disclosures / Credits

Consulted claude for recommendation on how to apply positional and conditional arguments using Kong framework.

@lizrabuya lizrabuya marked this pull request as ready for review December 15, 2025 22:11
@lizrabuya lizrabuya requested a review from a team as a code owner December 15, 2025 22:11
@mcncl mcncl merged commit ad21d23 into main Dec 15, 2025
1 check passed
@mcncl mcncl deleted the SUP-5774-package_push branch December 15, 2025 23:29
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