https://cloud.google.com/docs/terraform/best-practices-for-terraform?hl=jahttps://cloud.google.com/docs/terraform/best-practices-for-terraform?hl=ja

命名規則を採用する

ここは改善できるポイント。ベストプラクティスを尊重したい。

推奨:

resource "google_compute_instance" "web_server" {
  name = "web-server"
}

非推奨:

resource "google_compute_instance" "web-server" {
  name = "web-server"
}

これ、Google Cloud のリソース名がだいたいハイフン区切りなものなので、それに従ったほうがいいのかな、ということで一部を "google_compute_instance" "web-server" のようにしてしまっていた。アンダースコアでいいのね。物理名としてのリソースはハイフンがよさげだけど、Terraformの変数的にはアンダースコアで統一したほうがよさそうだ。

すべてのリソースの出力を公開する

これは疑問、そうなのか。でも、outputがないと、モジュールをどういう順番で組み立てればいいかわからないってことかな…？

環境ディレクトリに分割する

これ、できてる！すげえ！同僚に感謝！

ほかはざっくりできてそう、また見ていく。