Skip to content

Commit c91b2c7

Browse files
hcl2template: use diags.Extend instead of append
Since hcl.Diagnostics supports extending it through the `Extend' method, we might as well use it instead of manually appending a series of diagnostics to it.
1 parent a07bd93 commit c91b2c7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

hcl2template/types.packer_config.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -616,14 +616,14 @@ func (p *PackerConfig) InspectConfig(opts packer.InspectConfigOptions) int {
616616

617617
func (cfg *PackerConfig) Initialize(opts packer.InitializeOptions) hcl.Diagnostics {
618618
diags := cfg.InputVariables.ValidateValues()
619-
diags = append(diags, cfg.executeDatasources(opts.SkipDatasourcesExecution)...)
620-
diags = append(diags, checkForDuplicateLocalDefinition(cfg.LocalBlocks)...)
621-
diags = append(diags, cfg.evaluateLocalVariables()...)
619+
diags = diags.Extend(diags, checkForDuplicateLocalDefinition(cfg.LocalBlocks))
620+
diags = diags.Extend(cfg.executeDatasources(opts.SkipDatasourcesExecution))
621+
diags = diags.Extend(cfg.evaluateLocalVariables())
622622

623623
filterVarsFromLogs(cfg.InputVariables)
624624
filterVarsFromLogs(cfg.LocalVariables)
625625

626-
diags = append(diags, cfg.initializeBlocks()...)
626+
diags = diags.Extend(cfg.initializeBlocks())
627627

628628
return diags
629629
}

0 commit comments

Comments
 (0)