Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve user message #164

Merged
merged 1 commit into from
Apr 4, 2023
Merged

Improve user message #164

merged 1 commit into from
Apr 4, 2023

Conversation

marckhouzam
Copy link
Contributor

What this PR does / why we need it

The concept of caching is not mentioned in our documentation so we shouldl avoid complicating things for users by mentioning it in such a printout. Same goes for the plugin DB, which is an implementation detail.

Which issue(s) this PR fixes

Fixes # N/A

Describe testing done for PR

$ rm ~/.cache/_tanzu/plugin_inventory/
$ tz plugin search
[i] Reading plugin inventory for "projects.registry.vmware.com/tanzu_cli_stage/plugins/plugin-inventory:latest", this will take a few seconds.
  NAME     DESCRIPTION             TARGET  LATEST
  builder  Build Tanzu components  global  v0.1.0-dev-8-g6087ea00
  test     Test the CLI            global  v0.1.0-dev-8-g6087ea00

Release note

Additional information

Special notes for your reviewer

@marckhouzam marckhouzam requested a review from a team as a code owner April 3, 2023 23:41
@marckhouzam
Improve user message
926d119 
The concept of caching is not mentioned in our documentation so we
shouldl avoid complicating things for users by mentioning it in such
a printout.  Same goes for the plugin DB, which is an implementation
detail.

Signed-off-by: Marc Khouzam <kmarc@vmware.com>
@marckhouzam marckhouzam merged commit cd55b11 into vmware-tanzu:main Apr 4, 2023
@marckhouzam marckhouzam deleted the fix/cacheMsg branch April 4, 2023 12:52
@marckhouzam marckhouzam added this to the v0.90.0 milestone Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anujc25 anujc25 anujc25 approved these changes

Assignees
No one assigned
Labels
cla-not-required
Projects
None yet
Milestone
v0.90.0
Development

Successfully merging this pull request may close these issues.
3 participants
@marckhouzam @anujc25 @vmwclabot