Skip to content

Commit a6c9b5f

Browse files
committed
Address review suggestions regarding S404
1 parent c6cd747 commit a6c9b5f

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

ruff.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ select = ["ALL"]
1919
preview = true
2020
ignore = [
2121
"CPY001", # Skip copyright notice requirement at top of files
22-
"S404", # Allow importing 'subprocess' module to call external tools needed by these hooks
2322
]
2423

2524
[lint.isort]

src/pre_commit_terraform/terraform_docs_replace.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,11 @@
55
"""
66

77
import os
8-
import subprocess
8+
9+
# S404 - Allow importing 'subprocess' module to call external tools
10+
# needed by these hooks. FIXME - should be moved to separate module
11+
# when more hooks will be introduced
12+
import subprocess # noqa: S404
913
import warnings
1014
from argparse import ArgumentParser, Namespace
1115
from typing import cast as cast_to

0 commit comments

Comments
 (0)