gitignore icon indicating copy to clipboard operation
gitignore copied to clipboard

Update Terraform.gitignore to ignore graph files

Open RScrafted opened this issue 1 year ago • 1 comments

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

RScrafted avatar Jun 02 '24 16:06 RScrafted

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:

github-actions[bot] avatar Jun 10 '25 16:06 github-actions[bot]

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?

thorrsson avatar Jul 17 '25 19:07 thorrsson

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!

RScrafted avatar Jul 18 '25 10:07 RScrafted

Hi @thorrsson! Just wanted to check in — I believe I’ve addressed all requested changes, including:

  • Commenting out the *.dot line as optional, per the earliest feedback
  • Commenting out planout with a note so users can enable it if needed
  • Removing .terraform.lock.hcl from ignores based on Terraform’s recommendation
  • Removing the *.terraform.* pattern due to potential false positives
  • Removing .DS_Store to respect global gitignore best practices
  • Removing the overly broad *.out pattern to avoid unintended ignores
  • Removing .vscode folder ignore to keep workspace settings shareable

Please let me know if anything else needs attention. Thanks again for the review and support!

RScrafted avatar Jul 18 '25 11:07 RScrafted

Thanks @RScrafted

thorrsson avatar Jul 21 '25 18:07 thorrsson