Commit cd7d78b6 replaced use of

util.CheckErr(err) with if err != nil { return err } One replacement is incorrent. The current call of resource.NewBuilder is returned in r variable. Which was tested with util.CheckErr(r.Err()) before cd7d78b6 commit. It was replaced by if err != nil { return err } which is incorrect as err is not set by resource.NewBuilder call. The correct use is err := r.Err() if err != nil { return err }

Commit cd7d78b6 replaced use of
70c74fbe · Jan Chaloupka · 7e51e6ab · 70c74fbe
Commit 70c74fbe authored Sep 09, 2015 by Jan Chaloupka
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 0 deletions

get.go pkg/kubectl/cmd/get.go +1 -0

No files found.
--- a/pkg/kubectl/cmd/get.go
+++ b/pkg/kubectl/cmd/get.go
@@ -130,6 +130,7 @@ func RunGet(f *cmdutil.Factory, out io.Writer, cmd *cobra.Command, args []string
 			SingleResourceType().
 			Latest().
 			Do()
+		err := r.Err()
 		if err != nil {
 			return err
 		}