Update Terraform.gitignore to ignore graph files
Added line to ignore graph file('s).
Reasons for making this change:
I am doing a cloud deployment project. .gitignore is critical and relevant for me to avoid uploading unnecessary files. I find the template helpful and feel the necessity to ignore graph files. These are saved as .dot extension which is standard with terraform. I noticed the default template was missing graph ignore files which are optional in most cases as they serve as a local function. As a result, I have added the same.
Links to documentation supporting these rule changes:
https://developer.hashicorp.com/terraform/cli/commands/graph
If this is a new template:
- Link to application or project’s homepage: TODO
This PR is stale because there have been no updates in 90 days. It will close after 180 days of inactivity. Leave a comment if you want to keep it open :smile:
Personally I've committed the graph files to code bases before, @RScrafted would you be on board with commenting out the line but leaving it in the .gitignore file as optional with some commentary?
Personally I've committed the graph files to code bases before, @RScrafted would you be on board with commenting out the line but leaving it in the .gitignore file as optional with some commentary?
Thanks so much for your input! I’ve commented out the *.dot line and added a note so users can choose whether to ignore it based on their workflow. Really appreciate your insight!
Hi @thorrsson! Just wanted to check in — I believe I’ve addressed all requested changes, including:
- Commenting out the
*.dotline as optional, per the earliest feedback - Commenting out
planoutwith a note so users can enable it if needed - Removing
.terraform.lock.hclfrom ignores based on Terraform’s recommendation - Removing the
*.terraform.*pattern due to potential false positives - Removing
.DS_Storeto respect global gitignore best practices - Removing the overly broad
*.outpattern to avoid unintended ignores - Removing
.vscodefolder ignore to keep workspace settings shareable
Please let me know if anything else needs attention. Thanks again for the review and support!
Thanks @RScrafted