move this to after the prepare() is successful so that we don't try to run builds that didn't prepare successfully.

This commit is contained in:
Megan Marsh 2020-05-28 11:34:00 -07:00
parent 553d2a7d0b
commit 45be673c0f
1 changed files with 3 additions and 1 deletions

View File

@ -215,7 +215,6 @@ func (c *Core) GetBuilds(opts GetBuildsOptions) ([]Build, hcl.Diagnostics) {
})
continue
}
builds = append(builds, b)
// Now that build plugin has been launched, call Prepare()
log.Printf("Preparing build: %s", b.Name())
@ -233,6 +232,9 @@ func (c *Core) GetBuilds(opts GetBuildsOptions) ([]Build, hcl.Diagnostics) {
continue
}
// Only append builds to list if the Prepare() is successful.
builds = append(builds, b)
if len(warnings) > 0 {
for _, warning := range warnings {
diags = append(diags, &hcl.Diagnostic{