ref: rename inspect to show (#4927)

Per the Helm 3 plan, `helm inspect` and all of its subcommands have been moved to `helm show`.

Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>

cmd/helm/root.go

@@ -63,7 +63,7 @@ func newRootCmd(c helm.Interface, out io.Writer, args []string) *cobra.Command {
 		newCreateCmd(out),
 		newDependencyCmd(out),
 		newPullCmd(out),
-		newInspectCmd(out),
+		newShowCmd(out),
 		newLintCmd(out),
 		newPackageCmd(out),
 		newRepoCmd(out),

cmd/helm/show.go

@@ -29,19 +29,19 @@ import (
 	"k8s.io/helm/pkg/chart/loader"
 )
 
-const inspectDesc = `
+const showDesc = `
 This command inspects a chart and displays information. It takes a chart reference
 ('stable/drupal'), a full path to a directory or packaged chart, or a URL.
 
 Inspect prints the contents of the Chart.yaml file and the values.yaml file.
 `
 
-const inspectValuesDesc = `
+const showValuesDesc = `
 This command inspects a chart (directory, file, or URL) and displays the contents
 of the values.yaml file
 `
 
-const inspectChartDesc = `
+const showChartDesc = `
 This command inspects a chart (directory, file, or URL) and displays the contents
 of the Charts.yaml file
 `
@@ -51,7 +51,7 @@ This command inspects a chart (directory, file, or URL) and displays the content
 of the README file
 `
 
-type inspectOptions struct {
+type showOptions struct {
 	chartpath string
 	output    string
 
@@ -67,13 +67,14 @@ const (
 
 var readmeFileNames = []string{"readme.md", "readme.txt", "readme"}
 
-func newInspectCmd(out io.Writer) *cobra.Command {
-	o := &inspectOptions{output: all}
+func newShowCmd(out io.Writer) *cobra.Command {
+	o := &showOptions{output: all}
 
-	inspectCommand := &cobra.Command{
-		Use:   "inspect [CHART]",
+	showCommand := &cobra.Command{
+		Use:     "show [CHART]",
 		Short:   "inspect a chart",
-		Long:  inspectDesc,
+		Aliases: []string{"inspect"},
+		Long:    showDesc,
 		Args:    require.ExactArgs(1),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			cp, err := o.locateChart(args[0])
@@ -87,8 +88,8 @@ func newInspectCmd(out io.Writer) *cobra.Command {
 
 	valuesSubCmd := &cobra.Command{
 		Use:   "values [CHART]",
-		Short: "shows inspect values",
-		Long:  inspectValuesDesc,
+		Short: "shows values for this chart",
+		Long:  showValuesDesc,
 		Args:  require.ExactArgs(1),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			o.output = valuesOnly
@@ -103,8 +104,8 @@ func newInspectCmd(out io.Writer) *cobra.Command {
 
 	chartSubCmd := &cobra.Command{
 		Use:   "chart [CHART]",
-		Short: "shows inspect chart",
-		Long:  inspectChartDesc,
+		Short: "shows the chart",
+		Long:  showChartDesc,
 		Args:  require.ExactArgs(1),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			o.output = chartOnly
@@ -119,7 +120,7 @@ func newInspectCmd(out io.Writer) *cobra.Command {
 
 	readmeSubCmd := &cobra.Command{
 		Use:   "readme [CHART]",
-		Short: "shows inspect readme",
+		Short: "shows the chart's README",
 		Long:  readmeChartDesc,
 		Args:  require.ExactArgs(1),
 		RunE: func(cmd *cobra.Command, args []string) error {
@@ -133,19 +134,19 @@ func newInspectCmd(out io.Writer) *cobra.Command {
 		},
 	}
 
-	cmds := []*cobra.Command{inspectCommand, readmeSubCmd, valuesSubCmd, chartSubCmd}
+	cmds := []*cobra.Command{showCommand, readmeSubCmd, valuesSubCmd, chartSubCmd}
 	for _, subCmd := range cmds {
 		o.chartPathOptions.addFlags(subCmd.Flags())
 	}
 
 	for _, subCmd := range cmds[1:] {
-		inspectCommand.AddCommand(subCmd)
+		showCommand.AddCommand(subCmd)
 	}
 
-	return inspectCommand
+	return showCommand
 }
 
-func (i *inspectOptions) run(out io.Writer) error {
+func (i *showOptions) run(out io.Writer) error {
 	chrt, err := loader.Load(i.chartpath)
 	if err != nil {
 		return err

cmd/helm/show_test.go

@@ -23,10 +23,10 @@ import (
 	"testing"
 )
 
-func TestInspect(t *testing.T) {
+func TestShow(t *testing.T) {
 	b := bytes.NewBuffer(nil)
 
-	o := &inspectOptions{
+	o := &showOptions{
 		chartpath: "testdata/testcharts/alpine",
 		output:    all,
 	}
@@ -67,7 +67,7 @@ func TestInspect(t *testing.T) {
 
 	// Regression tests for missing values. See issue #1024.
 	b.Reset()
-	o = &inspectOptions{
+	o = &showOptions{
 		chartpath: "testdata/testcharts/novals",
 		output:    "values",
 	}