feature request: apply on merge

[atlantisbot]

Issue by @robertlabrie
Thursday Oct 19, 2017 at 12:21 GMT
Migrated from hootsuite/atlantis#162
Why was it migrated?

---

Would be great if merging a PR triggered a terraform apply

[atlantisbot]

Comment by @robertlabrie
Monday Oct 23, 2017 at 11:41 GMT

---

piling on my own request, plan on push would be dandy as well :)

[atlantisbot]

Comment by @lkysow
Monday Oct 23, 2017 at 13:42 GMT

---

Thanks for the ticket! The main reason we do apply on comment is that often there are errors when you run apply. If this happens then it's easier to push a change to an open pull request and run plan/apply again.

That being said, I don't think there's a reason we couldn't also support a final apply on merge for any plans that are outstanding. What do you think @anubhavmishra?

For plan on push, we definitely agree this is needed. I created #165 to track this separately.

[lkysow]

I've decided against doing this for now. Although there are valid use-cases for applying after merge, I'd like to focus on the most common use-cases where applying on the pull request and locking the directory until the pull request is merged works well.