Skip to content

Commit 2100afe

Browse files
committed
fix(style-guide): Adjust rule four for var validation to exclude bool vars
Signed-off-by: Tristan Jahnke <trjahnke@protonmail.com>
1 parent bb4b59f commit 2100afe

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

docs/terraform/style-guide.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ lowercase with only letters and underscores.
3434
for multiline detailed descriptions.
3535
3. Be explicit when it comes to variable types. If it is a map or object be diligent with marking that in the type along
3636
with its internal types.
37-
4. All variables should have at least one level of validation. Even if it is a simple variable that is always set in a
38-
vars file. Typos and mistakes happen. Validation catches that ahead of time.
37+
4. All variables should have at least one level of validation, excluding variables of type `bool`. Even if it is a
38+
simple variable that is always set in a vars file. Typos and mistakes happen. Validation catches that ahead of time.
3939
5. Create a condition that is as strict to the type and desired state of the variable without it becoming its own
4040
project to manage. Strict enough to catch real mistakes, loose enough not to require updating every time the
4141
upstream naming rules shift.

0 commit comments

Comments
 (0)