Skip to content

Commit 11d39bd

Browse files
owineclaudehappy-otter
committed
fix(scripts): add shellcheck suppressions for detect-critical-stacks
Add shellcheck disable directives with explanatory comments: - SC2034: OP_TOKEN validated but reserved for future use - SC2086: Intentional word splitting in printf for stack escaping All scripts now pass shellcheck validation. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <[email protected]> Co-Authored-By: Happy <[email protected]>
1 parent 0e383b4 commit 11d39bd

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

scripts/deployment/detect-critical-stacks.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ while [[ $# -gt 0 ]]; do
3434
shift 2
3535
;;
3636
--op-token)
37+
# shellcheck disable=SC2034 # Reserved for future use, validated for API consistency
3738
OP_TOKEN="$2"
3839
shift 2
3940
;;
@@ -54,6 +55,7 @@ log_info "Detecting critical stacks from compose file labels..."
5455
log_info "Scanning stacks: $STACKS"
5556

5657
# Escape stacks for positional argument
58+
# shellcheck disable=SC2086 # Word splitting intended - each stack becomes separate argument to printf
5759
STACKS_ESCAPED=$(printf '%q ' $STACKS)
5860

5961
# Execute detection script on remote server

0 commit comments

Comments
 (0)