search

buildkite / terraform-provider-buildkite

Terraform provider for Buildkite
https://registry.terraform.io/providers/buildkite/buildkite/latest
MIT License
56 stars 35 forks source link

Fix pipeline steps default on update #530

Closed drcapulet closed 5 months ago

drcapulet commented 6 months ago

The refactor in #474 didn't account for updates on pipelines that previously relied on the default setting of steps - this ensures it's always set if not otherwise provided.

jradtilbrook commented 6 months ago

Thanks for contributing this @drcapulet 👍 I've run the tests and they pass and will get the team to check this out further to get it merged in

jradtilbrook commented 5 months ago

@drcapulet I think these changes are correct but we found there was a larger problem with the way steps and pipeline templates interact. As a result, I've pulled these changes into #539 which resolves it all in one go. Thanks for you contribution!