Removed 'yaml' output from '--format' flag

Author: cmaglie

Diff for: .licenses/go/sigs.k8s.io/yaml.dep.yml

@@ -4,6 +4,10 @@ Here you can find a list of migration guides to handle breaking changes between
 
 ## 0.36.0
 
+### YAML output format is no more supported
+
+The `yaml` option of the `--format` flag is no more supported. Use `--format json` if machine parsable output is needed.
+
 ### The gRPC `cc.arduino.cli.commands.v1.CompileRequest.export_binaries` changed type.
 
 Previously the field `export_binaries` was a `google.protobuf.BoolValue`. We used this type because it expresses this

Diff for: .licenses/go/sigs.k8s.io/yaml/goyaml.v2.dep.yml

@@ -44,7 +44,6 @@ require (
 	google.golang.org/grpc v1.62.1
 	google.golang.org/protobuf v1.33.0
 	gopkg.in/yaml.v3 v3.0.1
-	sigs.k8s.io/yaml v1.4.0
 )
 
 require (

Diff for: docs/UPGRADING.md

@@ -82,7 +82,6 @@ github.com/golang/protobuf v1.5.3/go.mod h1:XVQd3VNwM+JqD3oG2Ue2ip4fOMUkwXdXDdiu
 github.com/google/go-cmp v0.3.0/go.mod h1:8QqcDgzrUqlUb/G2PQTWiueGozuR1884gddMywk6iLU=
 github.com/google/go-cmp v0.5.4/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE=
 github.com/google/go-cmp v0.5.5/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE=
-github.com/google/go-cmp v0.5.9/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
 github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
 github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
 github.com/h2non/filetype v1.1.3 h1:FKkx9QbD7HR/zjK1Ia5XiBsq9zdLi5Kf3zGyFTAFkGg=
@@ -300,5 +299,3 @@ gopkg.in/yaml.v2 v2.4.0/go.mod h1:RDklbk79AGWmwhnvt/jBztapEOGDOx6ZbXqjP6csGnQ=
 gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
 gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
 gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
-sigs.k8s.io/yaml v1.4.0 h1:Mk1wCc2gy/F0THH0TAp1QYyJNzRm2KCLy3o5ASXVI5E=
-sigs.k8s.io/yaml v1.4.0/go.mod h1:Ejl7/uTz7PSA4eKMyQCUTnhZYNmLIl+5c2lQPGR2BPY=

Diff for: go.mod

@@ -150,8 +150,8 @@ func createCliCommandTree(cmd *cobra.Command) {
 	cmd.RegisterFlagCompletionFunc("log-format", func(cmd *cobra.Command, args []string, toComplete string) ([]string, cobra.ShellCompDirective) {
 		return validLogFormats, cobra.ShellCompDirectiveDefault
 	})
-	validOutputFormats := []string{"text", "json", "jsonmini", "yaml"}
-	cmd.PersistentFlags().StringVar(&outputFormat, "format", "text", tr("The output format for the logs, can be: %s", strings.Join(validOutputFormats, ", ")))
+	validOutputFormats := []string{"text", "json", "jsonmini"}
+	cmd.PersistentFlags().StringVar(&outputFormat, "format", "text", tr("The command output format, can be: %s", strings.Join(validOutputFormats, ", ")))
 	cmd.RegisterFlagCompletionFunc("format", func(cmd *cobra.Command, args []string, toComplete string) ([]string, cobra.ShellCompDirective) {
 		return validOutputFormats, cobra.ShellCompDirectiveDefault
 	})

Diff for: go.sum

@@ -26,7 +26,6 @@ import (
 
 	"github.com/arduino/arduino-cli/internal/i18n"
 	"github.com/sirupsen/logrus"
-	"sigs.k8s.io/yaml"
 )
 
 // OutputFormat is an output format
@@ -39,14 +38,11 @@ const (
 	JSON
 	// MinifiedJSON format
 	MinifiedJSON
-	// YAML format
-	YAML
 )
 
 var formats = map[string]OutputFormat{
 	"json":     JSON,
 	"jsonmini": MinifiedJSON,
-	"yaml":     YAML,
 	"text":     Text,
 }
 
@@ -204,8 +200,6 @@ func Fatal(errorMsg string, exitCode ExitCode) {
 		d, _ = json.MarshalIndent(augment(res), "", "  ")
 	case MinifiedJSON:
 		d, _ = json.Marshal(augment(res))
-	case YAML:
-		d, _ = yaml.Marshal(augment(res))
 	default:
 		panic("unknown output format")
 	}
@@ -250,12 +244,6 @@ func PrintResult(res Result) {
 			Fatal(tr("Error during JSON encoding of the output: %v", err), ErrGeneric)
 		}
 		data = string(d)
-	case YAML:
-		d, err := yaml.Marshal(augment(res.Data()))
-		if err != nil {
-			Fatal(tr("Error during YAML encoding of the output: %v", err), ErrGeneric)
-		}
-		data = string(d)
 	case Text:
 		data = res.String()
 		if resErr, ok := res.(ErrorResult); ok {