Page MenuHomePhabricator

[scripts] Fix terraform lint
ClosedPublic

Authored by bartek on Nov 23 2023, 2:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 5, 3:24 AM
Unknown Object (File)
Tue, Jun 25, 10:44 AM
Unknown Object (File)
Tue, Jun 25, 10:44 AM
Unknown Object (File)
Tue, Jun 25, 10:44 AM
Unknown Object (File)
Tue, Jun 25, 10:35 AM
Unknown Object (File)
Mon, Jun 24, 10:55 AM
Unknown Object (File)
Thu, Jun 20, 11:54 PM
Unknown Object (File)
Thu, Jun 20, 3:26 PM
Subscribers

Details

Summary

When reviewing D9877 I realized that terraform format lint doesn't work in pre-commit hook. Fixed this.
Also changed it to auto-format instead of failing.

Test Plan

Made some formatting errors in terraform files, git staged them, and ran yarn lint-staged. It fixed them.

Output of terraform_pre_commit.sh (the modules/shared/dynamodb.tf was the file that was reformatted):

ᐅ ./scripts/terraform_pre_commit.sh
Formatting terraform...
modules/shared/dynamodb.tf
Validating 'dev' terraform configuration...
Success! The configuration is valid.

Validating 'remote' terraform configuration...
Success! The configuration is valid.

Diff Detail

Repository
rCOMM Comm
Branch
barthap/tf-lint
Lint
No Lint Coverage
Unit
No Test Coverage