Skip to content

Commit b873a3e

Browse files
committed
chore(launcher): Tidy shellcheck rules for IDE compatibility
shellcheck treats the shell=dash designation as indication to only care about dash rules; however IntelliJ idea will still highlight it as an issue. We are probably better to only suppress the relevant rule. Signed-off-by: Chad Wilson <29788154+chadlwilson@users.noreply.github.com> (cherry picked from commit e10eda1)
1 parent 164cb11 commit b873a3e

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

bin/jruby.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
#!/bin/sh
2-
# shellcheck shell=dash # local variable support
3-
# shellcheck disable=1007 # spurious warnings when initializing multiple vars
2+
# shellcheck disable=3043 # assume local variable support in shell
43

54
# -----------------------------------------------------------------------------
65
# jruby.sh - Start Script for the JRuby interpreter
@@ -59,6 +58,7 @@ fi
5958
# https://github.com/mentalisttraceur/esceval
6059
esceval()
6160
{
61+
# shellcheck disable=1007 # multiple empty var initialization intentional here
6262
local escaped= unescaped= output=
6363
REPLY=
6464

0 commit comments

Comments
 (0)