Skip to content

Merge branch 'main' into dudantas/town-crash #3353

Merge branch 'main' into dudantas/town-crash

Merge branch 'main' into dudantas/town-crash #3353

GitHub Actions / shellcheck succeeded Apr 17, 2024 in 1s

reviewdog [shellcheck] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (8)

start_gdb.sh|18 col 6| Check exit code directly with e.g. 'if mycmd;', not indirectly with $?.
docker/config.sh|13 col 11| Use grep -q instead of comparing output with [ -n .. ].
tests/build_and_run.sh|7 col 1| Use 'cd ... || exit' or 'cd ... || return' in case cd fails.
tests/build_and_run.sh|8 col 38| Quote this to prevent word splitting.
tests/build_and_run.sh|8 col 38| Use $(...) notation instead of legacy backticks ....
tests/build_and_run.sh|10 col 1| Use a ( subshell ) to avoid having to cd back.
tests/build_and_run.sh|11 col 1| Use 'cd ... || exit' or 'cd ... || return' in case cd fails.
recompile.sh|65 col 30| To redirect stdout+stderr, 2>&1 must be last (or use '{ cmd > file; } 2>&1' to clarify).