cagov / data-infrastructure

CalData infrastructure
https://cagov.github.io/data-infrastructure
MIT License
7 stars 0 forks source link

Update terraform CI to execute in its own job #398

Closed JamesSLogan closed 1 month ago

JamesSLogan commented 1 month ago

Fixes #319

This ended up being more painful than the related task in Caltrans' repo due to the terraform-docs automation. The terraform-docs action works for our use case, though it requires 1 invocation per file and also has a bug (see use of $OUTPUT_FILE here) that means it only fully works on 1 of our 3 files.

ian-r-rose commented 1 month ago

I'm also open to deleting the terraform-docs functionality, I haven't found it that useful. But if you figured it out, then that's great too!

JamesSLogan commented 1 month ago

I guess I would leave that up to you all! It should work either way...I did test the scenarios where files do get changed.