The main thrust of this change is to use the go convention of NewTHING functions to initialize the structs in internal/run instead of constructing struct literals in plan.go. Unfortunately, it'ss a big honkin' diff that touches almost every source file in the repo. It's probably easier to read in two chunks:
Alternatively, starting with the diffs for internal/helm/plan.go and internal/run/depupdate.go should give you a reasonably-clear overview of what's happened--plan.go shows how the run package's public interface has changed, while depupdate.go is a fairly short file that demonstrates the changes to the Steps.
Pre-merge checklist:
[x] Code changes have tests
[x] Any config changes are documented: (n/a)
If the change touches required config, there's a corresponding update to README.md
There's a corresponding update to docs/parameter_reference.md
For #67
The main thrust of this change is to use the go convention of
NewTHINGfunctions to initialize the structs ininternal/runinstead of constructing struct literals inplan.go. Unfortunately, it'ss a big honkin' diff that touches almost every source file in the repo. It's probably easier to read in two chunks:NewSTEPNAMEfunctionsrunandhelmpackagesAlternatively, starting with the diffs for
internal/helm/plan.goandinternal/run/depupdate.goshould give you a reasonably-clear overview of what's happened--plan.goshows how therunpackage's public interface has changed, whiledepupdate.gois a fairly short file that demonstrates the changes to the Steps.Pre-merge checklist:
README.mddocs/parameter_reference.md