Page MenuHomePhabricator

[scripts] Fix terraform lint
ClosedPublic

Authored by bartek on Nov 23 2023, 2:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 21, 5:34 AM
Unknown Object (File)
Fri, Apr 18, 4:18 AM
Unknown Object (File)
Fri, Apr 18, 3:05 AM
Unknown Object (File)
Thu, Apr 17, 8:30 PM
Unknown Object (File)
Thu, Apr 17, 11:04 AM
Unknown Object (File)
Fri, Apr 11, 8:14 PM
Unknown Object (File)
Wed, Apr 9, 1:23 AM
Unknown Object (File)
Tue, Apr 8, 10:41 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