mercari / tfnotify

A CLI command to parse Terraform execution result and notify it to GitHub
MIT License
620 stars 92 forks source link

try parsing pr number to create apply result comment on merged pr #66

Closed drlau closed 4 years ago

drlau commented 4 years ago

WHAT

Title

WHY

Right now, the apply result is commented on the commit just before the merge commit in master. If the PR merged in is not up to date with master, someone else's commit may be the commit right before the merge commit. This makes the apply result get commented on a completely unrelated commit instead of in the PR.

babarot commented 4 years ago

👀