Avoid rendering empty stats object when running diffs

egibs · egibs · commit b080ad92065e · 2024-12-30T16:36:53.000-06:00
Signed-off-by: egibs &lt;20933572+egibs@users.noreply.github.com&gt;
diff --git a/pkg/render/json.go b/pkg/render/json.go
@@ -52,7 +52,7 @@ func (r JSON) Full(_ context.Context, c *malcontent.Config, rep *malcontent.Repo
 		return true
 	})
 
-	if c != nil && c.Stats {
+	if c != nil && c.Stats && rep.Diff == nil {
 		jr.Stats = serializedStats(c, rep)
 	}
 
diff --git a/pkg/render/yaml.go b/pkg/render/yaml.go
@@ -52,7 +52,7 @@ func (r YAML) Full(_ context.Context, c *malcontent.Config, rep *malcontent.Repo
 		return true
 	})
 
-	if c != nil && c.Stats {
+	if c != nil && c.Stats && rep.Diff == nil {
 		yr.Stats = serializedStats(c, rep)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ func (r JSON) Full(_ context.Context, c malcontent.Config, rep malcontent.Repo`
`52`	`52`	`return true`
`53`	`53`	`})`
`54`	`54`
`55`		`- if c != nil && c.Stats {`
	`55`	`+ if c != nil && c.Stats && rep.Diff == nil {`
`56`	`56`	`jr.Stats = serializedStats(c, rep)`
`57`	`57`	`}`
`58`	`58`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ func (r YAML) Full(_ context.Context, c malcontent.Config, rep malcontent.Repo`
`52`	`52`	`return true`
`53`	`53`	`})`
`54`	`54`
`55`		`- if c != nil && c.Stats {`
	`55`	`+ if c != nil && c.Stats && rep.Diff == nil {`
`56`	`56`	`yr.Stats = serializedStats(c, rep)`
`57`	`57`	`}`
`58`	`58`