Skip to content
This repository was archived by the owner on May 18, 2026. It is now read-only.

Add new shorthands for some flags#398

Merged
warango4 merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue389
Oct 31, 2022
Merged

Add new shorthands for some flags#398
warango4 merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue389

Conversation

@warango4
Copy link
Copy Markdown
Contributor

Pull request

What this PR does / why we need it

Add new shorthands for flags

  • -a, --app (prior art - kctrl and kapp)
  • -e, --env (prior art - docker)
  • -i, --image
  • -l, --label (prior art - kubectl)
  • -p, --param
  • -t, --type

Which issue(s) this PR fixes

Fixes #389

Describe testing done for PR

Additional information or special notes for your reviewer

Signed-off-by: Wendy Arango warango@vmware.com

Signed-off-by: Wendy Arango <warango@vmware.com>
@warango4
Copy link
Copy Markdown
Contributor Author

image

@warango4
Copy link
Copy Markdown
Contributor Author

image

Copy link
Copy Markdown
Contributor

@odinnordico odinnordico left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@shaheerkootteeri shaheerkootteeri left a comment

Choose a reason for hiding this comment

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

LGTM

@warango4 warango4 merged commit 9d1a868 into vmware-tanzu:main Oct 31, 2022
@warango4 warango4 deleted the issue389 branch October 31, 2022 21:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Workload create/apply supports additional shorthand flag values

5 participants