terraform-aws-ec2-instance icon indicating copy to clipboard operation
terraform-aws-ec2-instance copied to clipboard

feat: Set module shortname for wrapper pre-commit hook

Open tofupup opened this issue 3 years ago β€’ 0 comments

Description

Add --module-repo-shortname argument to .pre-commit-config.yaml to help in running pre-commit checks (primarily from a docker container with the repo mounted).

Motivation and Context

Using the pre-commit-terraform docker container requires setting this variable to have wrappers/README.md properly generated.

Issue #295

Breaking Changes

No

How Has This Been Tested?

  • [ ] I have updated at least one of the examples/* to demonstrate and validate my change(s) Not applicable
  • [x] I have tested and validated these changes using one or more of the provided examples/* projects Not applicable, though I have verified that ec2_t2_unlimited still runs as expected I have tested this change by running pre-commit run -a directly, as well as using the latest pre-commit-terraform docker container, and the output is as expected, including properly setting source in wrappers/README.md
  • [x] I have executed pre-commit run -a on my pull request

tofupup avatar Sep 05 '22 23:09 tofupup