chore: add shellcheck disable to false positive of not reached

StrangeRanger · StrangeRanger · commit acf9dea38edd · 2026-05-21T11:44:10.000-07:00
diff --git a/hardening/UFW Cloudflare/ufw-cloudflare.bash b/hardening/UFW Cloudflare/ufw-cloudflare.bash
@@ -85,7 +85,9 @@ clean_exit() {
     exit "$exit_code"
 }
 
-# shellcheck disable=SC2329
+# shellcheck disable=SC2329,SC2317
+#   These appear to be false positives. The function is intended to be used in the 'ERR'
+#   trap handler, and the exit code is passed implicitly via the special variable '$?'.
 on_err() {
     local exit_code=$?
 

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,9 @@ clean_exit() {`
`85`	`85`	`exit "$exit_code"`
`86`	`86`	`}`
`87`	`87`
`88`		`-# shellcheck disable=SC2329`
	`88`	`+# shellcheck disable=SC2329,SC2317`
	`89`	`+# These appear to be false positives. The function is intended to be used in the 'ERR'`
	`90`	`+# trap handler, and the exit code is passed implicitly via the special variable '$?'.`
`89`	`91`	`on_err() {`
`90`	`92`	`local exit_code=$?`
`91`	`93`