Repository: helm/helm
Pull request #8557 (2020-08-05 11:23:44)

Fix Quick Start Guide Link in README.md
The previous link https://docs.helm.sh/using_helm/#quickstart-guide was redirecting into https://helm.sh/ja/docs/intro/ which is a Japanese version of the docs. I've fixed the link so it goes into the English version instead.

<!--  Thanks for sending a pull request!  Here are some tips for you:
1. Make sure to read the Contributing Guide before submitting your PR: https://github.com/helm/helm/blob/master/CONTRIBUTING.md
2. If this PR closes another issue, add 'closes #<issue number>' somewhere in the PR summary. GitHub will automatically close that issue when this PR gets merged. Alternatively, adding 'refs #<issue number>' will not close the issue, but help provide the reviewer more context.-->

**What this PR does / why we need it**:
This fixes the **Quick Start Guide** link in the **[Install](https://github.com/helm/helm/blob/master/README.md#Install)** section of [README.md](https://github.com/helm/helm/blob/master/README.md)
**Special notes for your reviewer**:
None
**If applicable**:
- [x] this PR contains documentation
- [ ] this PR contains unit tests
- [ ] this PR has been tested for backwards compatibility



Patch:
diff --git a/README.md b/README.md
index bb6908fdb..9c39175bc 100644
--- a/README.md
+++ b/README.md
@@ -43,7 +43,7 @@ If you want to use a package manager:
 - [GoFish](https://gofi.sh/) users can use `gofish install helm`.
 - [Snapcraft](https://snapcraft.io/) users can use `snap install helm --classic`
 
-To rapidly get Helm up and running, start with the [Quick Start Guide](https://docs.helm.sh/using_helm/#quickstart-guide).
+To rapidly get Helm up and running, start with the [Quick Start Guide](https://helm.sh/docs/intro/quickstart/).
 
 See the [installation guide](https://helm.sh/docs/intro/install/) for more options,
 including installing pre-releases.
diff --git a/cmd/helm/show.go b/cmd/helm/show.go
index fe5eaee26..c122d8476 100644
--- a/cmd/helm/show.go
+++ b/cmd/helm/show.go
@@ -151,9 +151,6 @@ func addShowFlags(subCmd *cobra.Command, client *action.Show) {
 	f := subCmd.Flags()
 
 	f.BoolVar(&client.Devel, "devel", false, "use development versions, too. Equivalent to version '>0.0.0-0'. If --version is set, this is ignored")
-	if subCmd.Name() == "values" {
-		f.StringVar(&client.JSONPathTemplate, "jsonpath", "", "supply a JSONPath expression to filter the output")
-	}
 	addChartPathOptionsFlags(f, &client.ChartPathOptions)
 
 	err := subCmd.RegisterFlagCompletionFunc("version", func(cmd *cobra.Command, args []string, toComplete string) ([]string, cobra.ShellCompDirective) {
diff --git a/pkg/action/show.go b/pkg/action/show.go
index 4eab2b4fd..9baa9cf43 100644
--- a/pkg/action/show.go
+++ b/pkg/action/show.go
@@ -20,8 +20,6 @@ import (
 	"fmt"
 	"strings"
 
-	"github.com/pkg/errors"
-	"k8s.io/cli-runtime/pkg/printers"
 	"sigs.k8s.io/yaml"
 
 	"helm.sh/helm/v3/pkg/chart"
@@ -54,10 +52,9 @@ func (o ShowOutputFormat) String() string {
 // It provides the implementation of 'helm show' and its respective subcommands.
 type Show struct {
 	ChartPathOptions
-	Devel            bool
-	OutputFormat     ShowOutputFormat
-	JSONPathTemplate string
-	chart            *chart.Chart // for testing
+	Devel        bool
+	OutputFormat ShowOutputFormat
+	chart        *chart.Chart // for testing
 }
 
 // NewShow creates a new Show object with the given configuration.
@@ -90,17 +87,9 @@ func (s *Show) Run(chartpath string) (string, error) {
 		if s.OutputFormat == ShowAll {
 			fmt.Fprintln(&out, "---")
 		}
-		if s.JSONPathTemplate != "" {
-			printer, err := printers.NewJSONPathPrinter(s.JSONPathTemplate)
-			if err != nil {
-				return "", errors.Wrapf(err, "error parsing jsonpath %s", s.JSONPathTemplate)
-			}
-			printer.Execute(&out, s.chart.Values)
-		} else {
-			for _, f := range s.chart.Raw {
-				if f.Name == chartutil.ValuesfileName {
-					fmt.Fprintln(&out, string(f.Data))
-				}
+		for _, f := range s.chart.Raw {
+			if f.Name == chartutil.ValuesfileName {
+				fmt.Fprintln(&out, string(f.Data))
 			}
 		}
 	}


Test patch:
diff --git a/pkg/action/show_test.go b/pkg/action/show_test.go
index a2efdc8ac..7be9daaa5 100644
--- a/pkg/action/show_test.go
+++ b/pkg/action/show_test.go
@@ -69,17 +69,3 @@ func TestShowNoValues(t *testing.T) {
 		t.Errorf("expected empty values buffer, got %s", output)
 	}
 }
-
-func TestShowValuesByJsonPathFormat(t *testing.T) {
-	client := NewShow(ShowValues)
-	client.JSONPathTemplate = "{$.nestedKey.simpleKey}"
-	client.chart = buildChart(withSampleValues())
-	output, err := client.Run("")
-	if err != nil {
-		t.Fatal(err)
-	}
-	expect := "simpleValue"
-	if output != expect {
-		t.Errorf("Expected\n%q\nGot\n%q\n", expect, output)
-	}
-}