Use plan instead of dry-run.

jmccann / drone-terraform

Drone plugin for triggering Terraform deployments

http://plugins.drone.io/jmccann/terraform/

Apache License 2.0

86 stars 93 forks source link

Closed jackspirou closed 8 years ago

jackspirou commented 8 years ago

Terraform users will be familiar with the terraform syntax and terminology, so we should use plan instead of dry-run. https://www.terraform.io/docs/commands/plan.html

If there are any other cases where the terraform terminology makes sense to use it, we should do that.

tboerger commented 8 years ago

It's a breaking change, but the plugin is documented only since yesterday. LGTM

jackspirou commented 8 years ago

LGTM