Skip to content

Commit 4d1dbf5

Browse files
committed
Add explanations for disabled linters
Signed-off-by: Timo Sand <[email protected]>
1 parent 623e1d3 commit 4d1dbf5

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

GNUmakefile

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,15 @@ lintcheck:
5050
tf-provider-lint:
5151
@branch=$$(git rev-parse --abbrev-ref HEAD); \
5252
printf "==> Running TF provider lint on branch: \033[1m%s\033[0m...\n" "🌿 $$branch 🌿";
53+
# Disabled linter rules:
54+
# AT001: TestCase missing CheckDestroy - not yet adopted across all tests
55+
# AT003: should use underscores in acc test names - not yet adopted across all tests
56+
# AT004: provider declaration should be omitted - intentionally kept for provider configuration tests
57+
# AT006: acc tests should not contain multiple resource.Test() invocations - not yet adopted across all tests
58+
# XAT001: acceptance test should use ErrorCheck - not all resources support destroy verification
59+
# XR003: resource should configure Timeouts - not yet adopted across all resources
60+
# XR007: avoid os/exec.Command - intentionally used for GitHub CLI integration and ssh-keygen in tests
61+
# XS002: schema should use keys in alphabetical order - not sure we want to enforce this
5362
tfproviderlintx \
5463
-AT001=false \
5564
-AT003=false \

0 commit comments

Comments
 (0)