From 44487feddb05ff75fdb5262b10e7f5bc1b276613 Mon Sep 17 00:00:00 2001 From: Glenn Jackman Date: Mon, 15 Jun 2026 08:05:09 -0400 Subject: [PATCH 1/4] shellcheck fixed for non-example exercise files --- .../practice/dnd-character/dnd_character.bats | 4 +- exercises/practice/list-ops/list_ops.bats | 70 +++++++++++-------- exercises/practice/markdown/markdown.bats | 2 +- .../palindrome-products/.meta/example2.sh | 2 +- .../pascals-triangle/.meta/example2.sh | 3 +- 5 files changed, 45 insertions(+), 36 deletions(-) diff --git a/exercises/practice/dnd-character/dnd_character.bats b/exercises/practice/dnd-character/dnd_character.bats index d3d0bc88..59e57ff0 100644 --- a/exercises/practice/dnd-character/dnd_character.bats +++ b/exercises/practice/dnd-character/dnd_character.bats @@ -153,8 +153,8 @@ between() { # random ability is within range @test "validate ability range and hitpoint value" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - for i in {1..50}; do - while read c v; do + for _ in {1..50}; do + while read -r c v; do if [[ $c == "hitpoints" ]]; then hits=$v else diff --git a/exercises/practice/list-ops/list_ops.bats b/exercises/practice/list-ops/list_ops.bats index 7f12d636..485e0b8a 100644 --- a/exercises/practice/list-ops/list_ops.bats +++ b/exercises/practice/list-ops/list_ops.bats @@ -1,4 +1,6 @@ #!/usr/bin/env bats +# shellcheck disable=SC2329 # "This function is never invoked." + load bats-extra # local version: 2.4.0.0 @@ -15,8 +17,8 @@ setup() { source list_ops.sh; } @test "append empty lists" { #[[ $BATS_RUN_SKIPPED == "true" ]] || skip - list1=() - list2=() + local list1=() + local list2=() list::append list1 "${list2[@]}" assert_equal "${#list1[@]}" 0 assert_equal "${list1[*]}" "" @@ -24,8 +26,8 @@ setup() { source list_ops.sh; } @test "append list to empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list1=() - list2=(1 2 3 4) + local list1=() + local list2=(1 2 3 4) list::append list1 "${list2[@]}" assert_equal "${#list1[@]}" 4 assert_equal "${list1[*]}" "1 2 3 4" @@ -33,8 +35,8 @@ setup() { source list_ops.sh; } @test "append empty list to list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list1=(1 2 3 4) - list2=() + local list1=(1 2 3 4) + local list2=() list::append list1 "${list2[@]}" assert_equal "${#list1[@]}" 4 assert_equal "${list1[*]}" "1 2 3 4" @@ -42,8 +44,8 @@ setup() { source list_ops.sh; } @test "append non-empty lists" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - l1=(1 2) - l2=(2 3 4 5) + local l1=(1 2) + local l2=(2 3 4 5) list::append l1 "${l2[@]}" assert_equal "${#l1[@]}" 6 assert_equal "${l1[*]}" "1 2 2 3 4 5" @@ -56,8 +58,8 @@ setup() { source list_ops.sh; } @test "filter empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=() - result=() + local list=() + local result=() isOdd () { (( $1 % 2 == 1 )); } list::filter isOdd list result assert_equal "${#result[@]}" 0 @@ -66,8 +68,8 @@ setup() { source list_ops.sh; } @test "filter non-empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(1 2 3 4 5) - result=() + local list=(1 2 3 4 5) + local result=() isOdd () { (( $1 % 2 == 1 )); } list::filter isOdd list result assert_equal "${#result[@]}" 3 @@ -82,8 +84,8 @@ setup() { source list_ops.sh; } @test "map empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=() - result=() + local list=() + local result=() incr () { echo $(( $1 + 1 )); } list::map incr list result assert_equal "${#result[@]}" ${#list[@]} @@ -92,8 +94,8 @@ setup() { source list_ops.sh; } @test "map non-empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(1 3 5 7) - result=() + local list=(1 3 5 7) + local result=() incr () { echo $(( $1 + 1 )); } list::map incr list result assert_equal "${#result[@]}" ${#list[@]} @@ -104,7 +106,8 @@ setup() { source list_ops.sh; } @test "foldl empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=() + local list=() + local result mult () { local acc=$1 elem=$2 echo $(( elem * acc )) @@ -115,7 +118,8 @@ setup() { source list_ops.sh; } @test "foldl direction independent function applied to non-empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(1 2 3 4) + local list=(1 2 3 4) + local result add () { local acc=$1 elem=$2 echo $(( elem + acc )) @@ -126,7 +130,8 @@ setup() { source list_ops.sh; } @test "foldl direction dependent function applied to non-empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(1 2 3 4) + local list=(1 2 3 4) + local result answer # For this test, we need a div function that performs # floating point arithmetic to preserve fractions. div () { @@ -141,7 +146,8 @@ setup() { source list_ops.sh; } # track-specific test @test "foldl not just numbers" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(H e l l o " " W o r l d "!") + local list=(H e l l o " " W o r l d "!") + local result concat () { local acc=$1 elem=$2 echo "${acc}${elem}" @@ -155,7 +161,8 @@ setup() { source list_ops.sh; } @test "foldr empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=() + local list=() + local result mult () { local elem=$1 acc=$2 echo $(( elem * acc )) @@ -166,7 +173,8 @@ setup() { source list_ops.sh; } @test "foldr direction independent function applied to non-empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(1 2 3 4) + local list=(1 2 3 4) + local result add () { local elem=$1 acc=$2 echo $(( elem + acc )) @@ -177,7 +185,8 @@ setup() { source list_ops.sh; } @test "foldr direction dependent function applied to non-empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(1 2 3 4) + local list=(1 2 3 4) + local answer result div () { local elem=$1 acc=$2 echo "$elem / $acc" | bc -l @@ -190,7 +199,8 @@ setup() { source list_ops.sh; } # track-specific test @test "foldr not just numbers" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(H e l l o " " W o r l d "!") + local list=(H e l l o " " W o r l d "!") + local result concat () { local elem=$1 acc=$2 echo "${acc}${elem}" @@ -203,8 +213,8 @@ setup() { source list_ops.sh; } @test "reverse empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=() - result=() + local list=() + local result=() list::reverse list result assert_equal "${#result[@]}" ${#list[@]} assert_equal "${result[*]}" "" @@ -212,8 +222,8 @@ setup() { source list_ops.sh; } @test "reverse non-empty list" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=(1 3 5 7) - result=() + local list=(1 3 5 7) + local result=() list::reverse list result assert_equal "${#result[@]}" ${#list[@]} assert_equal "${result[*]}" "7 5 3 1" @@ -221,8 +231,8 @@ setup() { source list_ops.sh; } @test "reverse with special characters" { [[ $BATS_RUN_SKIPPED == "true" ]] || skip - list=("R*" "l*") - result=() + local list=("R*" "l*") + local result=() list::reverse list result assert_equal "${#result[@]}" ${#list[@]} assert_equal "${result[*]}" "l* R*" diff --git a/exercises/practice/markdown/markdown.bats b/exercises/practice/markdown/markdown.bats index c638eb2e..9536f738 100644 --- a/exercises/practice/markdown/markdown.bats +++ b/exercises/practice/markdown/markdown.bats @@ -5,7 +5,7 @@ load bats-extra # uses external tool: mktemp -setup() { export MD_FILE=$( mktemp ); } +setup() { export MD_FILE; MD_FILE=$( mktemp ); } teardown() { rm -f "$MD_FILE"; } diff --git a/exercises/practice/palindrome-products/.meta/example2.sh b/exercises/practice/palindrome-products/.meta/example2.sh index e49ca673..31c2bff5 100644 --- a/exercises/practice/palindrome-products/.meta/example2.sh +++ b/exercises/practice/palindrome-products/.meta/example2.sh @@ -29,7 +29,7 @@ main() { for ((j = i; j <= max; j++)); do prod=$(( i * j )) if is_palindrome "$prod"; then - palindromes[$prod]+=" [$i, $j]" + palindromes[prod]+=" [$i, $j]" fi done done diff --git a/exercises/practice/pascals-triangle/.meta/example2.sh b/exercises/practice/pascals-triangle/.meta/example2.sh index b430f055..debd2a28 100644 --- a/exercises/practice/pascals-triangle/.meta/example2.sh +++ b/exercises/practice/pascals-triangle/.meta/example2.sh @@ -21,7 +21,6 @@ declare -i maxFactCached=1 main() { local -i n=$1 i - local spaces for (( i = 1; i <= n; i++ )); do # print the leading spaces for this row printf "%*s" $((n - i)) "" @@ -37,7 +36,7 @@ row() { _n=$(( n - 1 )) _k=$(( k - 1 )) binomialCoefficient $_n $_k - row+=( ${bcCache[$_n,$_k]} ) + row+=( "${bcCache[$_n,$_k]}" ) done echo "${row[*]}" } From 89cbe926c287a9f129ebe4ee2390ce5ffb9ae16d Mon Sep 17 00:00:00 2001 From: Glenn Jackman Date: Mon, 15 Jun 2026 08:15:15 -0400 Subject: [PATCH 2/4] shellcheck fixed for acronym/bats-extra.bash --- exercises/practice/acronym/bats-extra.bash | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/exercises/practice/acronym/bats-extra.bash b/exercises/practice/acronym/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/acronym/bats-extra.bash +++ b/exercises/practice/acronym/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ From 461309fbca53d82a6b49231ed7e2258daf488bf6 Mon Sep 17 00:00:00 2001 From: Glenn Jackman Date: Mon, 15 Jun 2026 08:16:45 -0400 Subject: [PATCH 3/4] copy acronym/bats-extra.bash to all other exercises --- exercises/practice/affine-cipher/bats-extra.bash | 7 ++++++- exercises/practice/all-your-base/bats-extra.bash | 7 ++++++- exercises/practice/allergies/bats-extra.bash | 7 ++++++- exercises/practice/anagram/bats-extra.bash | 7 ++++++- exercises/practice/armstrong-numbers/bats-extra.bash | 7 ++++++- exercises/practice/atbash-cipher/bats-extra.bash | 7 ++++++- exercises/practice/beer-song/bats-extra.bash | 7 ++++++- exercises/practice/binary-search/bats-extra.bash | 7 ++++++- exercises/practice/bob/bats-extra.bash | 7 ++++++- exercises/practice/bottle-song/bats-extra.bash | 7 ++++++- exercises/practice/bowling/bats-extra.bash | 7 ++++++- exercises/practice/change/bats-extra.bash | 7 ++++++- exercises/practice/clock/bats-extra.bash | 7 ++++++- exercises/practice/collatz-conjecture/bats-extra.bash | 7 ++++++- exercises/practice/crypto-square/bats-extra.bash | 7 ++++++- exercises/practice/darts/bats-extra.bash | 7 ++++++- exercises/practice/diamond/bats-extra.bash | 7 ++++++- exercises/practice/difference-of-squares/bats-extra.bash | 7 ++++++- exercises/practice/diffie-hellman/bats-extra.bash | 7 ++++++- exercises/practice/dnd-character/bats-extra.bash | 7 ++++++- exercises/practice/eliuds-eggs/bats-extra.bash | 7 ++++++- exercises/practice/error-handling/bats-extra.bash | 7 ++++++- exercises/practice/flower-field/bats-extra.bash | 7 ++++++- exercises/practice/food-chain/bats-extra.bash | 7 ++++++- exercises/practice/forth/bats-extra.bash | 7 ++++++- exercises/practice/gigasecond/bats-extra.bash | 7 ++++++- exercises/practice/grains/bats-extra.bash | 7 ++++++- exercises/practice/grep/bats-extra.bash | 7 ++++++- exercises/practice/hamming/bats-extra.bash | 7 ++++++- exercises/practice/hello-world/bats-extra.bash | 7 ++++++- exercises/practice/house/bats-extra.bash | 7 ++++++- exercises/practice/isbn-verifier/bats-extra.bash | 7 ++++++- exercises/practice/isogram/bats-extra.bash | 7 ++++++- exercises/practice/kindergarten-garden/bats-extra.bash | 7 ++++++- exercises/practice/knapsack/bats-extra.bash | 7 ++++++- exercises/practice/largest-series-product/bats-extra.bash | 7 ++++++- exercises/practice/leap/bats-extra.bash | 7 ++++++- exercises/practice/line-up/bats-extra.bash | 7 ++++++- exercises/practice/list-ops/bats-extra.bash | 7 ++++++- exercises/practice/luhn/bats-extra.bash | 7 ++++++- exercises/practice/markdown/bats-extra.bash | 7 ++++++- exercises/practice/matching-brackets/bats-extra.bash | 7 ++++++- exercises/practice/meetup/bats-extra.bash | 7 ++++++- exercises/practice/minesweeper/bats-extra.bash | 7 ++++++- exercises/practice/nth-prime/bats-extra.bash | 7 ++++++- exercises/practice/nucleotide-count/bats-extra.bash | 7 ++++++- exercises/practice/ocr-numbers/bats-extra.bash | 7 ++++++- exercises/practice/palindrome-products/bats-extra.bash | 7 ++++++- exercises/practice/pangram/bats-extra.bash | 7 ++++++- exercises/practice/pascals-triangle/bats-extra.bash | 7 ++++++- exercises/practice/perfect-numbers/bats-extra.bash | 7 ++++++- exercises/practice/phone-number/bats-extra.bash | 7 ++++++- exercises/practice/pig-latin/bats-extra.bash | 7 ++++++- exercises/practice/poker/bats-extra.bash | 7 ++++++- exercises/practice/prime-factors/bats-extra.bash | 7 ++++++- exercises/practice/protein-translation/bats-extra.bash | 7 ++++++- exercises/practice/proverb/bats-extra.bash | 7 ++++++- exercises/practice/pythagorean-triplet/bats-extra.bash | 7 ++++++- exercises/practice/queen-attack/bats-extra.bash | 7 ++++++- exercises/practice/rail-fence-cipher/bats-extra.bash | 7 ++++++- exercises/practice/raindrops/bats-extra.bash | 7 ++++++- exercises/practice/rational-numbers/bats-extra.bash | 7 ++++++- exercises/practice/rectangles/bats-extra.bash | 7 ++++++- exercises/practice/resistor-color-duo/bats-extra.bash | 7 ++++++- exercises/practice/resistor-color-trio/bats-extra.bash | 7 ++++++- exercises/practice/resistor-color/bats-extra.bash | 7 ++++++- exercises/practice/reverse-string/bats-extra.bash | 7 ++++++- exercises/practice/rna-transcription/bats-extra.bash | 7 ++++++- exercises/practice/robot-simulator/bats-extra.bash | 7 ++++++- exercises/practice/roman-numerals/bats-extra.bash | 7 ++++++- exercises/practice/rotational-cipher/bats-extra.bash | 7 ++++++- exercises/practice/run-length-encoding/bats-extra.bash | 7 ++++++- exercises/practice/satellite/bats-extra.bash | 7 ++++++- exercises/practice/say/bats-extra.bash | 7 ++++++- exercises/practice/scrabble-score/bats-extra.bash | 7 ++++++- exercises/practice/secret-handshake/bats-extra.bash | 7 ++++++- exercises/practice/series/bats-extra.bash | 7 ++++++- exercises/practice/sieve/bats-extra.bash | 7 ++++++- exercises/practice/simple-cipher/bats-extra.bash | 7 ++++++- exercises/practice/space-age/bats-extra.bash | 7 ++++++- exercises/practice/spiral-matrix/bats-extra.bash | 7 ++++++- exercises/practice/square-root/bats-extra.bash | 7 ++++++- exercises/practice/sublist/bats-extra.bash | 7 ++++++- exercises/practice/sum-of-multiples/bats-extra.bash | 7 ++++++- exercises/practice/swift-scheduling/bats-extra.bash | 7 ++++++- exercises/practice/tournament/bats-extra.bash | 7 ++++++- exercises/practice/transpose/bats-extra.bash | 7 ++++++- exercises/practice/triangle/bats-extra.bash | 7 ++++++- exercises/practice/twelve-days/bats-extra.bash | 7 ++++++- exercises/practice/two-bucket/bats-extra.bash | 7 ++++++- exercises/practice/two-fer/bats-extra.bash | 7 ++++++- .../practice/variable-length-quantity/bats-extra.bash | 7 ++++++- exercises/practice/word-count/bats-extra.bash | 7 ++++++- exercises/practice/wordy/bats-extra.bash | 7 ++++++- exercises/practice/yacht/bats-extra.bash | 7 ++++++- 95 files changed, 570 insertions(+), 95 deletions(-) diff --git a/exercises/practice/affine-cipher/bats-extra.bash b/exercises/practice/affine-cipher/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/affine-cipher/bats-extra.bash +++ b/exercises/practice/affine-cipher/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/all-your-base/bats-extra.bash b/exercises/practice/all-your-base/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/all-your-base/bats-extra.bash +++ b/exercises/practice/all-your-base/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/allergies/bats-extra.bash b/exercises/practice/allergies/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/allergies/bats-extra.bash +++ b/exercises/practice/allergies/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/anagram/bats-extra.bash b/exercises/practice/anagram/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/anagram/bats-extra.bash +++ b/exercises/practice/anagram/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/armstrong-numbers/bats-extra.bash b/exercises/practice/armstrong-numbers/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/armstrong-numbers/bats-extra.bash +++ b/exercises/practice/armstrong-numbers/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/atbash-cipher/bats-extra.bash b/exercises/practice/atbash-cipher/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/atbash-cipher/bats-extra.bash +++ b/exercises/practice/atbash-cipher/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/beer-song/bats-extra.bash b/exercises/practice/beer-song/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/beer-song/bats-extra.bash +++ b/exercises/practice/beer-song/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/binary-search/bats-extra.bash b/exercises/practice/binary-search/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/binary-search/bats-extra.bash +++ b/exercises/practice/binary-search/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/bob/bats-extra.bash b/exercises/practice/bob/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/bob/bats-extra.bash +++ b/exercises/practice/bob/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/bottle-song/bats-extra.bash b/exercises/practice/bottle-song/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/bottle-song/bats-extra.bash +++ b/exercises/practice/bottle-song/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/bowling/bats-extra.bash b/exercises/practice/bowling/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/bowling/bats-extra.bash +++ b/exercises/practice/bowling/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/change/bats-extra.bash b/exercises/practice/change/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/change/bats-extra.bash +++ b/exercises/practice/change/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/clock/bats-extra.bash b/exercises/practice/clock/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/clock/bats-extra.bash +++ b/exercises/practice/clock/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/collatz-conjecture/bats-extra.bash b/exercises/practice/collatz-conjecture/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/collatz-conjecture/bats-extra.bash +++ b/exercises/practice/collatz-conjecture/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/crypto-square/bats-extra.bash b/exercises/practice/crypto-square/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/crypto-square/bats-extra.bash +++ b/exercises/practice/crypto-square/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/darts/bats-extra.bash b/exercises/practice/darts/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/darts/bats-extra.bash +++ b/exercises/practice/darts/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/diamond/bats-extra.bash b/exercises/practice/diamond/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/diamond/bats-extra.bash +++ b/exercises/practice/diamond/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/difference-of-squares/bats-extra.bash b/exercises/practice/difference-of-squares/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/difference-of-squares/bats-extra.bash +++ b/exercises/practice/difference-of-squares/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/diffie-hellman/bats-extra.bash b/exercises/practice/diffie-hellman/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/diffie-hellman/bats-extra.bash +++ b/exercises/practice/diffie-hellman/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/dnd-character/bats-extra.bash b/exercises/practice/dnd-character/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/dnd-character/bats-extra.bash +++ b/exercises/practice/dnd-character/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/eliuds-eggs/bats-extra.bash b/exercises/practice/eliuds-eggs/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/eliuds-eggs/bats-extra.bash +++ b/exercises/practice/eliuds-eggs/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/error-handling/bats-extra.bash b/exercises/practice/error-handling/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/error-handling/bats-extra.bash +++ b/exercises/practice/error-handling/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/flower-field/bats-extra.bash b/exercises/practice/flower-field/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/flower-field/bats-extra.bash +++ b/exercises/practice/flower-field/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/food-chain/bats-extra.bash b/exercises/practice/food-chain/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/food-chain/bats-extra.bash +++ b/exercises/practice/food-chain/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/forth/bats-extra.bash b/exercises/practice/forth/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/forth/bats-extra.bash +++ b/exercises/practice/forth/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/gigasecond/bats-extra.bash b/exercises/practice/gigasecond/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/gigasecond/bats-extra.bash +++ b/exercises/practice/gigasecond/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/grains/bats-extra.bash b/exercises/practice/grains/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/grains/bats-extra.bash +++ b/exercises/practice/grains/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/grep/bats-extra.bash b/exercises/practice/grep/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/grep/bats-extra.bash +++ b/exercises/practice/grep/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/hamming/bats-extra.bash b/exercises/practice/hamming/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/hamming/bats-extra.bash +++ b/exercises/practice/hamming/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/hello-world/bats-extra.bash b/exercises/practice/hello-world/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/hello-world/bats-extra.bash +++ b/exercises/practice/hello-world/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/house/bats-extra.bash b/exercises/practice/house/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/house/bats-extra.bash +++ b/exercises/practice/house/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/isbn-verifier/bats-extra.bash b/exercises/practice/isbn-verifier/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/isbn-verifier/bats-extra.bash +++ b/exercises/practice/isbn-verifier/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/isogram/bats-extra.bash b/exercises/practice/isogram/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/isogram/bats-extra.bash +++ b/exercises/practice/isogram/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/kindergarten-garden/bats-extra.bash b/exercises/practice/kindergarten-garden/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/kindergarten-garden/bats-extra.bash +++ b/exercises/practice/kindergarten-garden/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/knapsack/bats-extra.bash b/exercises/practice/knapsack/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/knapsack/bats-extra.bash +++ b/exercises/practice/knapsack/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/largest-series-product/bats-extra.bash b/exercises/practice/largest-series-product/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/largest-series-product/bats-extra.bash +++ b/exercises/practice/largest-series-product/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/leap/bats-extra.bash b/exercises/practice/leap/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/leap/bats-extra.bash +++ b/exercises/practice/leap/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/line-up/bats-extra.bash b/exercises/practice/line-up/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/line-up/bats-extra.bash +++ b/exercises/practice/line-up/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/list-ops/bats-extra.bash b/exercises/practice/list-ops/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/list-ops/bats-extra.bash +++ b/exercises/practice/list-ops/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/luhn/bats-extra.bash b/exercises/practice/luhn/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/luhn/bats-extra.bash +++ b/exercises/practice/luhn/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/markdown/bats-extra.bash b/exercises/practice/markdown/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/markdown/bats-extra.bash +++ b/exercises/practice/markdown/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/matching-brackets/bats-extra.bash b/exercises/practice/matching-brackets/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/matching-brackets/bats-extra.bash +++ b/exercises/practice/matching-brackets/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/meetup/bats-extra.bash b/exercises/practice/meetup/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/meetup/bats-extra.bash +++ b/exercises/practice/meetup/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/minesweeper/bats-extra.bash b/exercises/practice/minesweeper/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/minesweeper/bats-extra.bash +++ b/exercises/practice/minesweeper/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/nth-prime/bats-extra.bash b/exercises/practice/nth-prime/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/nth-prime/bats-extra.bash +++ b/exercises/practice/nth-prime/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/nucleotide-count/bats-extra.bash b/exercises/practice/nucleotide-count/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/nucleotide-count/bats-extra.bash +++ b/exercises/practice/nucleotide-count/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/ocr-numbers/bats-extra.bash b/exercises/practice/ocr-numbers/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/ocr-numbers/bats-extra.bash +++ b/exercises/practice/ocr-numbers/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/palindrome-products/bats-extra.bash b/exercises/practice/palindrome-products/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/palindrome-products/bats-extra.bash +++ b/exercises/practice/palindrome-products/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/pangram/bats-extra.bash b/exercises/practice/pangram/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/pangram/bats-extra.bash +++ b/exercises/practice/pangram/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/pascals-triangle/bats-extra.bash b/exercises/practice/pascals-triangle/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/pascals-triangle/bats-extra.bash +++ b/exercises/practice/pascals-triangle/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/perfect-numbers/bats-extra.bash b/exercises/practice/perfect-numbers/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/perfect-numbers/bats-extra.bash +++ b/exercises/practice/perfect-numbers/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/phone-number/bats-extra.bash b/exercises/practice/phone-number/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/phone-number/bats-extra.bash +++ b/exercises/practice/phone-number/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/pig-latin/bats-extra.bash b/exercises/practice/pig-latin/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/pig-latin/bats-extra.bash +++ b/exercises/practice/pig-latin/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/poker/bats-extra.bash b/exercises/practice/poker/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/poker/bats-extra.bash +++ b/exercises/practice/poker/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/prime-factors/bats-extra.bash b/exercises/practice/prime-factors/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/prime-factors/bats-extra.bash +++ b/exercises/practice/prime-factors/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/protein-translation/bats-extra.bash b/exercises/practice/protein-translation/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/protein-translation/bats-extra.bash +++ b/exercises/practice/protein-translation/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/proverb/bats-extra.bash b/exercises/practice/proverb/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/proverb/bats-extra.bash +++ b/exercises/practice/proverb/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/pythagorean-triplet/bats-extra.bash b/exercises/practice/pythagorean-triplet/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/pythagorean-triplet/bats-extra.bash +++ b/exercises/practice/pythagorean-triplet/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/queen-attack/bats-extra.bash b/exercises/practice/queen-attack/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/queen-attack/bats-extra.bash +++ b/exercises/practice/queen-attack/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/rail-fence-cipher/bats-extra.bash b/exercises/practice/rail-fence-cipher/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/rail-fence-cipher/bats-extra.bash +++ b/exercises/practice/rail-fence-cipher/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/raindrops/bats-extra.bash b/exercises/practice/raindrops/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/raindrops/bats-extra.bash +++ b/exercises/practice/raindrops/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/rational-numbers/bats-extra.bash b/exercises/practice/rational-numbers/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/rational-numbers/bats-extra.bash +++ b/exercises/practice/rational-numbers/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/rectangles/bats-extra.bash b/exercises/practice/rectangles/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/rectangles/bats-extra.bash +++ b/exercises/practice/rectangles/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/resistor-color-duo/bats-extra.bash b/exercises/practice/resistor-color-duo/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/resistor-color-duo/bats-extra.bash +++ b/exercises/practice/resistor-color-duo/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/resistor-color-trio/bats-extra.bash b/exercises/practice/resistor-color-trio/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/resistor-color-trio/bats-extra.bash +++ b/exercises/practice/resistor-color-trio/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/resistor-color/bats-extra.bash b/exercises/practice/resistor-color/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/resistor-color/bats-extra.bash +++ b/exercises/practice/resistor-color/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/reverse-string/bats-extra.bash b/exercises/practice/reverse-string/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/reverse-string/bats-extra.bash +++ b/exercises/practice/reverse-string/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/rna-transcription/bats-extra.bash b/exercises/practice/rna-transcription/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/rna-transcription/bats-extra.bash +++ b/exercises/practice/rna-transcription/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/robot-simulator/bats-extra.bash b/exercises/practice/robot-simulator/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/robot-simulator/bats-extra.bash +++ b/exercises/practice/robot-simulator/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/roman-numerals/bats-extra.bash b/exercises/practice/roman-numerals/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/roman-numerals/bats-extra.bash +++ b/exercises/practice/roman-numerals/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/rotational-cipher/bats-extra.bash b/exercises/practice/rotational-cipher/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/rotational-cipher/bats-extra.bash +++ b/exercises/practice/rotational-cipher/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/run-length-encoding/bats-extra.bash b/exercises/practice/run-length-encoding/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/run-length-encoding/bats-extra.bash +++ b/exercises/practice/run-length-encoding/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/satellite/bats-extra.bash b/exercises/practice/satellite/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/satellite/bats-extra.bash +++ b/exercises/practice/satellite/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/say/bats-extra.bash b/exercises/practice/say/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/say/bats-extra.bash +++ b/exercises/practice/say/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/scrabble-score/bats-extra.bash b/exercises/practice/scrabble-score/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/scrabble-score/bats-extra.bash +++ b/exercises/practice/scrabble-score/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/secret-handshake/bats-extra.bash b/exercises/practice/secret-handshake/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/secret-handshake/bats-extra.bash +++ b/exercises/practice/secret-handshake/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/series/bats-extra.bash b/exercises/practice/series/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/series/bats-extra.bash +++ b/exercises/practice/series/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/sieve/bats-extra.bash b/exercises/practice/sieve/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/sieve/bats-extra.bash +++ b/exercises/practice/sieve/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/simple-cipher/bats-extra.bash b/exercises/practice/simple-cipher/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/simple-cipher/bats-extra.bash +++ b/exercises/practice/simple-cipher/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/space-age/bats-extra.bash b/exercises/practice/space-age/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/space-age/bats-extra.bash +++ b/exercises/practice/space-age/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/spiral-matrix/bats-extra.bash b/exercises/practice/spiral-matrix/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/spiral-matrix/bats-extra.bash +++ b/exercises/practice/spiral-matrix/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/square-root/bats-extra.bash b/exercises/practice/square-root/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/square-root/bats-extra.bash +++ b/exercises/practice/square-root/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/sublist/bats-extra.bash b/exercises/practice/sublist/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/sublist/bats-extra.bash +++ b/exercises/practice/sublist/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/sum-of-multiples/bats-extra.bash b/exercises/practice/sum-of-multiples/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/sum-of-multiples/bats-extra.bash +++ b/exercises/practice/sum-of-multiples/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/swift-scheduling/bats-extra.bash b/exercises/practice/swift-scheduling/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/swift-scheduling/bats-extra.bash +++ b/exercises/practice/swift-scheduling/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/tournament/bats-extra.bash b/exercises/practice/tournament/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/tournament/bats-extra.bash +++ b/exercises/practice/tournament/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/transpose/bats-extra.bash b/exercises/practice/transpose/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/transpose/bats-extra.bash +++ b/exercises/practice/transpose/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/triangle/bats-extra.bash b/exercises/practice/triangle/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/triangle/bats-extra.bash +++ b/exercises/practice/triangle/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/twelve-days/bats-extra.bash b/exercises/practice/twelve-days/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/twelve-days/bats-extra.bash +++ b/exercises/practice/twelve-days/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/two-bucket/bats-extra.bash b/exercises/practice/two-bucket/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/two-bucket/bats-extra.bash +++ b/exercises/practice/two-bucket/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/two-fer/bats-extra.bash b/exercises/practice/two-fer/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/two-fer/bats-extra.bash +++ b/exercises/practice/two-fer/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/variable-length-quantity/bats-extra.bash b/exercises/practice/variable-length-quantity/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/variable-length-quantity/bats-extra.bash +++ b/exercises/practice/variable-length-quantity/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/word-count/bats-extra.bash b/exercises/practice/word-count/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/word-count/bats-extra.bash +++ b/exercises/practice/word-count/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/wordy/bats-extra.bash b/exercises/practice/wordy/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/wordy/bats-extra.bash +++ b/exercises/practice/wordy/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ diff --git a/exercises/practice/yacht/bats-extra.bash b/exercises/practice/yacht/bats-extra.bash index 54d48070..3b370fac 100644 --- a/exercises/practice/yacht/bats-extra.bash +++ b/exercises/practice/yacht/bats-extra.bash @@ -20,7 +20,9 @@ # . # +# shellcheck disable=SC2120 # (warning): fail references arguments, but none are ever passed. fail() { + # shellcheck disable=2015 # (info): Note that A && B || C is not if-then-else. C may run when A is true. (( $# == 0 )) && batslib_err || batslib_err "$@" return 1 } @@ -119,7 +121,7 @@ batslib_print_kv_single_or_multi() { else local -i i for (( i=1; i < ${#pairs[@]}; i+=2 )); do - pairs[$i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" + pairs[i]="$( batslib_prefix < <(printf '%s' "${pairs[$i]}") )" done batslib_print_kv_multi "${pairs[@]}" fi @@ -136,6 +138,7 @@ batslib_prefix() { batslib_mark() { local -r symbol="$1"; shift # Sort line numbers. + # shellcheck disable=SC2046 # (warning): Quote this to prevent word splitting. set -- $( sort -nu <<< "$( printf '%d\n' "$@" )" ) local line @@ -366,6 +369,8 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then + # glennj: I don't know what command's status `$?` is at this point + # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. if [[ '' =~ $expected ]] || (( $? == 2 )); then echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ From 6ad9abf44d2bb7172b3d1ae92171bedfe7fb14b5 Mon Sep 17 00:00:00 2001 From: Glenn Jackman Date: Mon, 15 Jun 2026 12:26:39 -0400 Subject: [PATCH 4/4] update comments in bats-extra.bash files --- exercises/practice/acronym/bats-extra.bash | 4 ++-- exercises/practice/affine-cipher/bats-extra.bash | 4 ++-- exercises/practice/all-your-base/bats-extra.bash | 4 ++-- exercises/practice/allergies/bats-extra.bash | 4 ++-- exercises/practice/anagram/bats-extra.bash | 4 ++-- exercises/practice/armstrong-numbers/bats-extra.bash | 4 ++-- exercises/practice/atbash-cipher/bats-extra.bash | 4 ++-- exercises/practice/beer-song/bats-extra.bash | 4 ++-- exercises/practice/binary-search/bats-extra.bash | 4 ++-- exercises/practice/bob/bats-extra.bash | 4 ++-- exercises/practice/bottle-song/bats-extra.bash | 4 ++-- exercises/practice/bowling/bats-extra.bash | 4 ++-- exercises/practice/change/bats-extra.bash | 4 ++-- exercises/practice/clock/bats-extra.bash | 4 ++-- exercises/practice/collatz-conjecture/bats-extra.bash | 4 ++-- exercises/practice/crypto-square/bats-extra.bash | 4 ++-- exercises/practice/darts/bats-extra.bash | 4 ++-- exercises/practice/diamond/bats-extra.bash | 4 ++-- exercises/practice/difference-of-squares/bats-extra.bash | 4 ++-- exercises/practice/diffie-hellman/bats-extra.bash | 4 ++-- exercises/practice/dnd-character/bats-extra.bash | 4 ++-- exercises/practice/eliuds-eggs/bats-extra.bash | 4 ++-- exercises/practice/error-handling/bats-extra.bash | 4 ++-- exercises/practice/flower-field/bats-extra.bash | 4 ++-- exercises/practice/food-chain/bats-extra.bash | 4 ++-- exercises/practice/forth/bats-extra.bash | 4 ++-- exercises/practice/gigasecond/bats-extra.bash | 4 ++-- exercises/practice/grains/bats-extra.bash | 4 ++-- exercises/practice/grep/bats-extra.bash | 4 ++-- exercises/practice/hamming/bats-extra.bash | 4 ++-- exercises/practice/hello-world/bats-extra.bash | 4 ++-- exercises/practice/house/bats-extra.bash | 4 ++-- exercises/practice/isbn-verifier/bats-extra.bash | 4 ++-- exercises/practice/isogram/bats-extra.bash | 4 ++-- exercises/practice/kindergarten-garden/bats-extra.bash | 4 ++-- exercises/practice/knapsack/bats-extra.bash | 4 ++-- exercises/practice/largest-series-product/bats-extra.bash | 4 ++-- exercises/practice/leap/bats-extra.bash | 4 ++-- exercises/practice/line-up/bats-extra.bash | 4 ++-- exercises/practice/list-ops/bats-extra.bash | 4 ++-- exercises/practice/luhn/bats-extra.bash | 4 ++-- exercises/practice/markdown/bats-extra.bash | 4 ++-- exercises/practice/matching-brackets/bats-extra.bash | 4 ++-- exercises/practice/meetup/bats-extra.bash | 4 ++-- exercises/practice/minesweeper/bats-extra.bash | 4 ++-- exercises/practice/nth-prime/bats-extra.bash | 4 ++-- exercises/practice/nucleotide-count/bats-extra.bash | 4 ++-- exercises/practice/ocr-numbers/bats-extra.bash | 4 ++-- exercises/practice/palindrome-products/bats-extra.bash | 4 ++-- exercises/practice/pangram/bats-extra.bash | 4 ++-- exercises/practice/pascals-triangle/bats-extra.bash | 4 ++-- exercises/practice/perfect-numbers/bats-extra.bash | 4 ++-- exercises/practice/phone-number/bats-extra.bash | 4 ++-- exercises/practice/pig-latin/bats-extra.bash | 4 ++-- exercises/practice/poker/bats-extra.bash | 4 ++-- exercises/practice/prime-factors/bats-extra.bash | 4 ++-- exercises/practice/protein-translation/bats-extra.bash | 4 ++-- exercises/practice/proverb/bats-extra.bash | 4 ++-- exercises/practice/pythagorean-triplet/bats-extra.bash | 4 ++-- exercises/practice/queen-attack/bats-extra.bash | 4 ++-- exercises/practice/rail-fence-cipher/bats-extra.bash | 4 ++-- exercises/practice/raindrops/bats-extra.bash | 4 ++-- exercises/practice/rational-numbers/bats-extra.bash | 4 ++-- exercises/practice/rectangles/bats-extra.bash | 4 ++-- exercises/practice/resistor-color-duo/bats-extra.bash | 4 ++-- exercises/practice/resistor-color-trio/bats-extra.bash | 4 ++-- exercises/practice/resistor-color/bats-extra.bash | 4 ++-- exercises/practice/reverse-string/bats-extra.bash | 4 ++-- exercises/practice/rna-transcription/bats-extra.bash | 4 ++-- exercises/practice/robot-simulator/bats-extra.bash | 4 ++-- exercises/practice/roman-numerals/bats-extra.bash | 4 ++-- exercises/practice/rotational-cipher/bats-extra.bash | 4 ++-- exercises/practice/run-length-encoding/bats-extra.bash | 4 ++-- exercises/practice/satellite/bats-extra.bash | 4 ++-- exercises/practice/say/bats-extra.bash | 4 ++-- exercises/practice/scrabble-score/bats-extra.bash | 4 ++-- exercises/practice/secret-handshake/bats-extra.bash | 4 ++-- exercises/practice/series/bats-extra.bash | 4 ++-- exercises/practice/sieve/bats-extra.bash | 4 ++-- exercises/practice/simple-cipher/bats-extra.bash | 4 ++-- exercises/practice/space-age/bats-extra.bash | 4 ++-- exercises/practice/spiral-matrix/bats-extra.bash | 4 ++-- exercises/practice/square-root/bats-extra.bash | 4 ++-- exercises/practice/sublist/bats-extra.bash | 4 ++-- exercises/practice/sum-of-multiples/bats-extra.bash | 4 ++-- exercises/practice/swift-scheduling/bats-extra.bash | 4 ++-- exercises/practice/tournament/bats-extra.bash | 4 ++-- exercises/practice/transpose/bats-extra.bash | 4 ++-- exercises/practice/triangle/bats-extra.bash | 4 ++-- exercises/practice/twelve-days/bats-extra.bash | 4 ++-- exercises/practice/two-bucket/bats-extra.bash | 4 ++-- exercises/practice/two-fer/bats-extra.bash | 4 ++-- exercises/practice/variable-length-quantity/bats-extra.bash | 4 ++-- exercises/practice/word-count/bats-extra.bash | 4 ++-- exercises/practice/wordy/bats-extra.bash | 4 ++-- exercises/practice/yacht/bats-extra.bash | 4 ++-- 96 files changed, 192 insertions(+), 192 deletions(-) diff --git a/exercises/practice/acronym/bats-extra.bash b/exercises/practice/acronym/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/acronym/bats-extra.bash +++ b/exercises/practice/acronym/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/affine-cipher/bats-extra.bash b/exercises/practice/affine-cipher/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/affine-cipher/bats-extra.bash +++ b/exercises/practice/affine-cipher/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/all-your-base/bats-extra.bash b/exercises/practice/all-your-base/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/all-your-base/bats-extra.bash +++ b/exercises/practice/all-your-base/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/allergies/bats-extra.bash b/exercises/practice/allergies/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/allergies/bats-extra.bash +++ b/exercises/practice/allergies/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/anagram/bats-extra.bash b/exercises/practice/anagram/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/anagram/bats-extra.bash +++ b/exercises/practice/anagram/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/armstrong-numbers/bats-extra.bash b/exercises/practice/armstrong-numbers/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/armstrong-numbers/bats-extra.bash +++ b/exercises/practice/armstrong-numbers/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/atbash-cipher/bats-extra.bash b/exercises/practice/atbash-cipher/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/atbash-cipher/bats-extra.bash +++ b/exercises/practice/atbash-cipher/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/beer-song/bats-extra.bash b/exercises/practice/beer-song/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/beer-song/bats-extra.bash +++ b/exercises/practice/beer-song/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/binary-search/bats-extra.bash b/exercises/practice/binary-search/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/binary-search/bats-extra.bash +++ b/exercises/practice/binary-search/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/bob/bats-extra.bash b/exercises/practice/bob/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/bob/bats-extra.bash +++ b/exercises/practice/bob/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/bottle-song/bats-extra.bash b/exercises/practice/bottle-song/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/bottle-song/bats-extra.bash +++ b/exercises/practice/bottle-song/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/bowling/bats-extra.bash b/exercises/practice/bowling/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/bowling/bats-extra.bash +++ b/exercises/practice/bowling/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/change/bats-extra.bash b/exercises/practice/change/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/change/bats-extra.bash +++ b/exercises/practice/change/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/clock/bats-extra.bash b/exercises/practice/clock/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/clock/bats-extra.bash +++ b/exercises/practice/clock/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/collatz-conjecture/bats-extra.bash b/exercises/practice/collatz-conjecture/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/collatz-conjecture/bats-extra.bash +++ b/exercises/practice/collatz-conjecture/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/crypto-square/bats-extra.bash b/exercises/practice/crypto-square/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/crypto-square/bats-extra.bash +++ b/exercises/practice/crypto-square/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/darts/bats-extra.bash b/exercises/practice/darts/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/darts/bats-extra.bash +++ b/exercises/practice/darts/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/diamond/bats-extra.bash b/exercises/practice/diamond/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/diamond/bats-extra.bash +++ b/exercises/practice/diamond/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/difference-of-squares/bats-extra.bash b/exercises/practice/difference-of-squares/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/difference-of-squares/bats-extra.bash +++ b/exercises/practice/difference-of-squares/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/diffie-hellman/bats-extra.bash b/exercises/practice/diffie-hellman/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/diffie-hellman/bats-extra.bash +++ b/exercises/practice/diffie-hellman/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/dnd-character/bats-extra.bash b/exercises/practice/dnd-character/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/dnd-character/bats-extra.bash +++ b/exercises/practice/dnd-character/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/eliuds-eggs/bats-extra.bash b/exercises/practice/eliuds-eggs/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/eliuds-eggs/bats-extra.bash +++ b/exercises/practice/eliuds-eggs/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/error-handling/bats-extra.bash b/exercises/practice/error-handling/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/error-handling/bats-extra.bash +++ b/exercises/practice/error-handling/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/flower-field/bats-extra.bash b/exercises/practice/flower-field/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/flower-field/bats-extra.bash +++ b/exercises/practice/flower-field/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/food-chain/bats-extra.bash b/exercises/practice/food-chain/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/food-chain/bats-extra.bash +++ b/exercises/practice/food-chain/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/forth/bats-extra.bash b/exercises/practice/forth/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/forth/bats-extra.bash +++ b/exercises/practice/forth/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/gigasecond/bats-extra.bash b/exercises/practice/gigasecond/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/gigasecond/bats-extra.bash +++ b/exercises/practice/gigasecond/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/grains/bats-extra.bash b/exercises/practice/grains/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/grains/bats-extra.bash +++ b/exercises/practice/grains/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/grep/bats-extra.bash b/exercises/practice/grep/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/grep/bats-extra.bash +++ b/exercises/practice/grep/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/hamming/bats-extra.bash b/exercises/practice/hamming/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/hamming/bats-extra.bash +++ b/exercises/practice/hamming/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/hello-world/bats-extra.bash b/exercises/practice/hello-world/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/hello-world/bats-extra.bash +++ b/exercises/practice/hello-world/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/house/bats-extra.bash b/exercises/practice/house/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/house/bats-extra.bash +++ b/exercises/practice/house/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/isbn-verifier/bats-extra.bash b/exercises/practice/isbn-verifier/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/isbn-verifier/bats-extra.bash +++ b/exercises/practice/isbn-verifier/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/isogram/bats-extra.bash b/exercises/practice/isogram/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/isogram/bats-extra.bash +++ b/exercises/practice/isogram/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/kindergarten-garden/bats-extra.bash b/exercises/practice/kindergarten-garden/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/kindergarten-garden/bats-extra.bash +++ b/exercises/practice/kindergarten-garden/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/knapsack/bats-extra.bash b/exercises/practice/knapsack/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/knapsack/bats-extra.bash +++ b/exercises/practice/knapsack/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/largest-series-product/bats-extra.bash b/exercises/practice/largest-series-product/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/largest-series-product/bats-extra.bash +++ b/exercises/practice/largest-series-product/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/leap/bats-extra.bash b/exercises/practice/leap/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/leap/bats-extra.bash +++ b/exercises/practice/leap/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/line-up/bats-extra.bash b/exercises/practice/line-up/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/line-up/bats-extra.bash +++ b/exercises/practice/line-up/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/list-ops/bats-extra.bash b/exercises/practice/list-ops/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/list-ops/bats-extra.bash +++ b/exercises/practice/list-ops/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/luhn/bats-extra.bash b/exercises/practice/luhn/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/luhn/bats-extra.bash +++ b/exercises/practice/luhn/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/markdown/bats-extra.bash b/exercises/practice/markdown/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/markdown/bats-extra.bash +++ b/exercises/practice/markdown/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/matching-brackets/bats-extra.bash b/exercises/practice/matching-brackets/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/matching-brackets/bats-extra.bash +++ b/exercises/practice/matching-brackets/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/meetup/bats-extra.bash b/exercises/practice/meetup/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/meetup/bats-extra.bash +++ b/exercises/practice/meetup/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/minesweeper/bats-extra.bash b/exercises/practice/minesweeper/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/minesweeper/bats-extra.bash +++ b/exercises/practice/minesweeper/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/nth-prime/bats-extra.bash b/exercises/practice/nth-prime/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/nth-prime/bats-extra.bash +++ b/exercises/practice/nth-prime/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/nucleotide-count/bats-extra.bash b/exercises/practice/nucleotide-count/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/nucleotide-count/bats-extra.bash +++ b/exercises/practice/nucleotide-count/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/ocr-numbers/bats-extra.bash b/exercises/practice/ocr-numbers/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/ocr-numbers/bats-extra.bash +++ b/exercises/practice/ocr-numbers/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/palindrome-products/bats-extra.bash b/exercises/practice/palindrome-products/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/palindrome-products/bats-extra.bash +++ b/exercises/practice/palindrome-products/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/pangram/bats-extra.bash b/exercises/practice/pangram/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/pangram/bats-extra.bash +++ b/exercises/practice/pangram/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/pascals-triangle/bats-extra.bash b/exercises/practice/pascals-triangle/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/pascals-triangle/bats-extra.bash +++ b/exercises/practice/pascals-triangle/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/perfect-numbers/bats-extra.bash b/exercises/practice/perfect-numbers/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/perfect-numbers/bats-extra.bash +++ b/exercises/practice/perfect-numbers/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/phone-number/bats-extra.bash b/exercises/practice/phone-number/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/phone-number/bats-extra.bash +++ b/exercises/practice/phone-number/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/pig-latin/bats-extra.bash b/exercises/practice/pig-latin/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/pig-latin/bats-extra.bash +++ b/exercises/practice/pig-latin/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/poker/bats-extra.bash b/exercises/practice/poker/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/poker/bats-extra.bash +++ b/exercises/practice/poker/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/prime-factors/bats-extra.bash b/exercises/practice/prime-factors/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/prime-factors/bats-extra.bash +++ b/exercises/practice/prime-factors/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/protein-translation/bats-extra.bash b/exercises/practice/protein-translation/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/protein-translation/bats-extra.bash +++ b/exercises/practice/protein-translation/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/proverb/bats-extra.bash b/exercises/practice/proverb/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/proverb/bats-extra.bash +++ b/exercises/practice/proverb/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/pythagorean-triplet/bats-extra.bash b/exercises/practice/pythagorean-triplet/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/pythagorean-triplet/bats-extra.bash +++ b/exercises/practice/pythagorean-triplet/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/queen-attack/bats-extra.bash b/exercises/practice/queen-attack/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/queen-attack/bats-extra.bash +++ b/exercises/practice/queen-attack/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/rail-fence-cipher/bats-extra.bash b/exercises/practice/rail-fence-cipher/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/rail-fence-cipher/bats-extra.bash +++ b/exercises/practice/rail-fence-cipher/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/raindrops/bats-extra.bash b/exercises/practice/raindrops/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/raindrops/bats-extra.bash +++ b/exercises/practice/raindrops/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/rational-numbers/bats-extra.bash b/exercises/practice/rational-numbers/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/rational-numbers/bats-extra.bash +++ b/exercises/practice/rational-numbers/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/rectangles/bats-extra.bash b/exercises/practice/rectangles/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/rectangles/bats-extra.bash +++ b/exercises/practice/rectangles/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/resistor-color-duo/bats-extra.bash b/exercises/practice/resistor-color-duo/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/resistor-color-duo/bats-extra.bash +++ b/exercises/practice/resistor-color-duo/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/resistor-color-trio/bats-extra.bash b/exercises/practice/resistor-color-trio/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/resistor-color-trio/bats-extra.bash +++ b/exercises/practice/resistor-color-trio/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/resistor-color/bats-extra.bash b/exercises/practice/resistor-color/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/resistor-color/bats-extra.bash +++ b/exercises/practice/resistor-color/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/reverse-string/bats-extra.bash b/exercises/practice/reverse-string/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/reverse-string/bats-extra.bash +++ b/exercises/practice/reverse-string/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/rna-transcription/bats-extra.bash b/exercises/practice/rna-transcription/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/rna-transcription/bats-extra.bash +++ b/exercises/practice/rna-transcription/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/robot-simulator/bats-extra.bash b/exercises/practice/robot-simulator/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/robot-simulator/bats-extra.bash +++ b/exercises/practice/robot-simulator/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/roman-numerals/bats-extra.bash b/exercises/practice/roman-numerals/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/roman-numerals/bats-extra.bash +++ b/exercises/practice/roman-numerals/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/rotational-cipher/bats-extra.bash b/exercises/practice/rotational-cipher/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/rotational-cipher/bats-extra.bash +++ b/exercises/practice/rotational-cipher/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/run-length-encoding/bats-extra.bash b/exercises/practice/run-length-encoding/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/run-length-encoding/bats-extra.bash +++ b/exercises/practice/run-length-encoding/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/satellite/bats-extra.bash b/exercises/practice/satellite/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/satellite/bats-extra.bash +++ b/exercises/practice/satellite/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/say/bats-extra.bash b/exercises/practice/say/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/say/bats-extra.bash +++ b/exercises/practice/say/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/scrabble-score/bats-extra.bash b/exercises/practice/scrabble-score/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/scrabble-score/bats-extra.bash +++ b/exercises/practice/scrabble-score/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/secret-handshake/bats-extra.bash b/exercises/practice/secret-handshake/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/secret-handshake/bats-extra.bash +++ b/exercises/practice/secret-handshake/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/series/bats-extra.bash b/exercises/practice/series/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/series/bats-extra.bash +++ b/exercises/practice/series/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/sieve/bats-extra.bash b/exercises/practice/sieve/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/sieve/bats-extra.bash +++ b/exercises/practice/sieve/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/simple-cipher/bats-extra.bash b/exercises/practice/simple-cipher/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/simple-cipher/bats-extra.bash +++ b/exercises/practice/simple-cipher/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/space-age/bats-extra.bash b/exercises/practice/space-age/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/space-age/bats-extra.bash +++ b/exercises/practice/space-age/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/spiral-matrix/bats-extra.bash b/exercises/practice/spiral-matrix/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/spiral-matrix/bats-extra.bash +++ b/exercises/practice/spiral-matrix/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/square-root/bats-extra.bash b/exercises/practice/square-root/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/square-root/bats-extra.bash +++ b/exercises/practice/square-root/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/sublist/bats-extra.bash b/exercises/practice/sublist/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/sublist/bats-extra.bash +++ b/exercises/practice/sublist/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/sum-of-multiples/bats-extra.bash b/exercises/practice/sum-of-multiples/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/sum-of-multiples/bats-extra.bash +++ b/exercises/practice/sum-of-multiples/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/swift-scheduling/bats-extra.bash b/exercises/practice/swift-scheduling/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/swift-scheduling/bats-extra.bash +++ b/exercises/practice/swift-scheduling/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/tournament/bats-extra.bash b/exercises/practice/tournament/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/tournament/bats-extra.bash +++ b/exercises/practice/tournament/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/transpose/bats-extra.bash b/exercises/practice/transpose/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/transpose/bats-extra.bash +++ b/exercises/practice/transpose/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/triangle/bats-extra.bash b/exercises/practice/triangle/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/triangle/bats-extra.bash +++ b/exercises/practice/triangle/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/twelve-days/bats-extra.bash b/exercises/practice/twelve-days/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/twelve-days/bats-extra.bash +++ b/exercises/practice/twelve-days/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/two-bucket/bats-extra.bash b/exercises/practice/two-bucket/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/two-bucket/bats-extra.bash +++ b/exercises/practice/two-bucket/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/two-fer/bats-extra.bash b/exercises/practice/two-fer/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/two-fer/bats-extra.bash +++ b/exercises/practice/two-fer/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/variable-length-quantity/bats-extra.bash b/exercises/practice/variable-length-quantity/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/variable-length-quantity/bats-extra.bash +++ b/exercises/practice/variable-length-quantity/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/word-count/bats-extra.bash b/exercises/practice/word-count/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/word-count/bats-extra.bash +++ b/exercises/practice/word-count/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/wordy/bats-extra.bash b/exercises/practice/wordy/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/wordy/bats-extra.bash +++ b/exercises/practice/wordy/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail diff --git a/exercises/practice/yacht/bats-extra.bash b/exercises/practice/yacht/bats-extra.bash index 3b370fac..7cef63ea 100644 --- a/exercises/practice/yacht/bats-extra.bash +++ b/exercises/practice/yacht/bats-extra.bash @@ -369,9 +369,9 @@ assert_output() { | fail fi elif (( is_mode_regexp )); then - # glennj: I don't know what command's status `$?` is at this point - # shellcheck disable=SC2319 # (warning): This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten. + # shellcheck disable=SC2319 # (warning) `$?` in a conditional expression: https://github.com/koalaman/shellcheck/wiki/SC2319 if [[ '' =~ $expected ]] || (( $? == 2 )); then + # the regex matches an empty string or it is syntactically incorrect. echo "Invalid extended regular expression: \`$expected'" \ | batslib_decorate 'ERROR: assert_output' \ | fail