Page MenuHomePhabricator

[scripts] Fix terraform lint
ClosedPublic

Authored by bartek on Nov 23 2023, 2:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 5, 1:22 PM
Unknown Object (File)
Sat, Dec 28, 3:39 PM
Unknown Object (File)
Mon, Dec 23, 11:36 AM
Unknown Object (File)
Mon, Dec 23, 11:36 AM
Unknown Object (File)
Mon, Dec 23, 11:35 AM
Unknown Object (File)
Mon, Dec 23, 11:26 AM
Unknown Object (File)
Mon, Dec 16, 6:37 PM
Unknown Object (File)
Sat, Dec 14, 5:49 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