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

Add INFO message pointing to new csc get command#155

Merged
shashwathi merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue151
Jun 3, 2022
Merged

Add INFO message pointing to new csc get command#155
shashwathi merged 1 commit into
vmware-tanzu:mainfrom
warango4:issue151

Conversation

@warango4
Copy link
Copy Markdown
Contributor

@warango4 warango4 commented Jun 2, 2022

Pull request

What this PR does / why we need it

Add info message in csc list command to make users aware of new csc get command

Which issue(s) this PR fixes

Fixes #151

Describe testing done for PR

Additional information or special notes for your reviewer

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

@warango4
Copy link
Copy Markdown
Contributor Author

warango4 commented Jun 2, 2022

image

Comment thread pkg/commands/clustersupplychain_list.go Outdated
@warango4 warango4 requested review from rashedkvm and shashwathi June 2, 2022 21:14
Add info message in csc list command to make users aware of new csc get command

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

to be consistent with the other INFO messages, we should:

  1. remove the word "INFO"
  2. add a new line after the info message

So it looks like this:

06/03/22-8:13:50 apps-cli-plugin %tanzu apps cluster-supply-chain list
NAME                 READY   AGE
basic-image-to-url   Ready   82m
source-to-url        Ready   82m

View supply chain details by running "tanzu apps cluster-supply-chain get <name>"

06/03/22-8:13:52 apps-cli-plugin %|

notice the empty new line following the info message

@warango4
Copy link
Copy Markdown
Contributor Author

warango4 commented Jun 3, 2022

image

@heyjcollins
Copy link
Copy Markdown
Contributor

perfect now.

@shashwathi shashwathi merged commit 779a8f0 into vmware-tanzu:main Jun 3, 2022
@warango4 warango4 deleted the issue151 branch June 6, 2022 14:59
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.

tanzu apps cluster-supply-chain get <name> command is recommended at the end of the output from cluster-supply-chain list

6 participants