answerdigital / terraform-modules

The repo for the infrastructure as code
MIT License
3 stars 4 forks source link

Fix logic for listing modules #58

Closed cmbuckley closed 1 year ago

cmbuckley commented 1 year ago

Logic for listing all modules includes the test directories within modules, which fails the workflow and creates unnecessary entries in dependabot.yml.

Decided to use -maxdepth 2, which does enforce the folder structure somewhat, but that's already the case because the changed_modules list looks at modules/*/*/*.tf. So we can solve both of those together if we ever need to.

Alternatives I considered that don't work:

robg-test commented 1 year ago

I somewhat wonder if we'll ever need a Maxdepth more than 2.

E.g. Variations on EC2 / DB etc.

Happy for now