test(npm-globals): silence SC2016 on the literal jq pin check

Merged
SH
16d ago
+1-0

Summary

  • #3080 merged before its shellcheck follow-up landed, so shell-lint on main flags SC2016 on the literal jq program in spec/npm_globals_spec.sh. The $r there is jq syntax, not shell expansion, so this adds a scoped # shellcheck disable=SC2016.

Verification

  • shellcheck spec/npm_globals_spec.sh is clean.
  • shellspec spec/npm_globals_spec.sh: 76 examples, 0 failures.

decided: scoped disable directive over rewriting the jq check through bash -c escaping - the program is literal jq and the directive keeps it readable

šŸ¤– Generated with Claude Code


Summary by cubic

Silences SC2016 for the literal jq program in spec/npm_globals_spec.sh, which was broken on main after #3080 merged before its shellcheck follow-up. The $r in that check is jq syntax, not shell expansion, so a scoped # shellcheck disable=SC2016 keeps the directive readable without rewriting the jq program.