mercari / tfnotify

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

Fix region user defined substitution format #103

Closed KeisukeYamashita closed 2 years ago

KeisukeYamashita commented 2 years ago

WHAT

I have added _ (understore) before the variable.

WHY

Because it doesn't follow the format. REGION is a custom substitution which the Cloud Build doesn't build-in support.🙏

Substitutions must begin with an underscore () and use only uppercase-letters and numbers (respecting the regular expression [A-Z0-9_]+)

Ref: https://cloud.google.com/build/docs/configuring-builds/substitute-variable-values