search

MobilityData / mobility-feed-api

Apache License 2.0
9 stars 3 forks source link

Review naming convention in terraform #802

Open cka-y opened 1 week ago

cka-y commented 1 week ago

_Originally posted by @jcpitre in https://github.com/MobilityData/mobility-feed-api/pull/801#discussion_r1834817201_

jcpitre commented 5 days ago

Some of the resources in terraform have a name in the resource line and another with in the resource (e.g. main.tf). This is confusing, as evidenced by the #801 bug.

We should have a way to distinguish the name in the resource line, which AFAIK is only used within terraform, and the explicit name in the resource. Maybe the terraform one should have a _tf suffix?