Skip to content

Commit 9d5d843

Browse files
committed
fix: prefer 2 space indent
yaml.v3 changed the default indent to 4 spaces, see: go-yaml/yaml@3c39489 in order to avoid unnecessary whitespace changes in code previously formatted with argo_image_updater, use a 2-space indent instead.
1 parent 91b359f commit 9d5d843

File tree

2 files changed

+127
-114
lines changed

2 files changed

+127
-114
lines changed

pkg/argocd/update.go

Lines changed: 22 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package argocd
22

33
import (
4+
"bytes"
45
"context"
56
"fmt"
67
"path/filepath"
@@ -418,6 +419,20 @@ func setAppImage(app *v1alpha1.Application, img *image.ContainerImage) error {
418419
return err
419420
}
420421

422+
func marshalWithIndent(in interface{}, indent int) (out []byte, err error) {
423+
var b bytes.Buffer
424+
encoder := yaml.NewEncoder(&b)
425+
defer encoder.Close()
426+
encoder.SetIndent(indent)
427+
if err = encoder.Encode(in); err != nil {
428+
return nil, err
429+
}
430+
if err = encoder.Close(); err != nil {
431+
return nil, err
432+
}
433+
return b.Bytes(), nil
434+
}
435+
421436
// marshalParamsOverride marshals the parameter overrides of a given application
422437
// into YAML bytes
423438
func marshalParamsOverride(app *v1alpha1.Application, originalData []byte) ([]byte, error) {
@@ -441,16 +456,16 @@ func marshalParamsOverride(app *v1alpha1.Application, originalData []byte) ([]by
441456
}
442457

443458
if len(originalData) == 0 {
444-
override, err = yaml.Marshal(newParams)
459+
override, err = marshalWithIndent(newParams, 2)
445460
break
446461
}
447462
err = yaml.Unmarshal(originalData, &params)
448463
if err != nil {
449-
override, err = yaml.Marshal(newParams)
464+
override, err = marshalWithIndent(newParams, 2)
450465
break
451466
}
452467
mergeKustomizeOverride(&params, &newParams)
453-
override, err = yaml.Marshal(params)
468+
override, err = marshalWithIndent(params, 2)
454469
case ApplicationTypeHelm:
455470
if appSource.Helm == nil {
456471
return []byte{}, nil
@@ -505,7 +520,7 @@ func marshalParamsOverride(app *v1alpha1.Application, originalData []byte) ([]by
505520
}
506521
}
507522

508-
override, err = yaml.Marshal(&helmNewValues)
523+
override, err = marshalWithIndent(&helmNewValues, 2)
509524
} else {
510525
var params helmOverride
511526
newParams := helmOverride{
@@ -518,16 +533,16 @@ func marshalParamsOverride(app *v1alpha1.Application, originalData []byte) ([]by
518533
log.WithContext().AddField("application", app).Debugf("values: '%s'", outputParams)
519534

520535
if len(originalData) == 0 {
521-
override, err = yaml.Marshal(newParams)
536+
override, err = marshalWithIndent(newParams, 2)
522537
break
523538
}
524539
err = yaml.Unmarshal(originalData, &params)
525540
if err != nil {
526-
override, err = yaml.Marshal(newParams)
541+
override, err = marshalWithIndent(newParams, 2)
527542
break
528543
}
529544
mergeHelmOverride(&params, &newParams)
530-
override, err = yaml.Marshal(params)
545+
override, err = marshalWithIndent(params, 2)
531546
}
532547
default:
533548
err = fmt.Errorf("unsupported application type")

0 commit comments

Comments
 (0)