Page MenuHomePhabricator

[scripts] Fix terraform lint
ClosedPublic

Authored by bartek on Nov 23 2023, 2:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 20, 11:53 PM
Unknown Object (File)
Tue, Jan 28, 5:56 PM
Unknown Object (File)
Tue, Jan 28, 5:56 PM
Unknown Object (File)
Tue, Jan 28, 5:56 PM
Unknown Object (File)
Tue, Jan 28, 5:56 PM
Unknown Object (File)
Jan 21 2025, 7:11 AM
Unknown Object (File)
Jan 16 2025, 8:00 PM
Unknown Object (File)
Jan 15 2025, 3:14 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable