Page MenuHomePhabricator

[scripts] Fix terraform lint
ClosedPublic

Authored by bartek on Nov 23 2023, 2:44 AM.
Tags
None
Referenced Files
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
Unknown Object (File)
Thu, Jun 13, 7:07 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable