Skip to content

Commit

Permalink
Add some comments explaining shellcheck disables
Browse files Browse the repository at this point in the history
  • Loading branch information
WalterKolczynski-NOAA committed Oct 25, 2022
1 parent c814838 commit 521cf30
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion sorc/build_all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ while getopts ":a:c:hov" option; do
c) _partial_opt+="-c ${OPTARG} ";;
h) _usage;;
o) _ops_opt+="-o";;
# s) _build_ufs_opt+="-s ${OPTARG} ";;
v) _verbose_opt="-v";;
:)
echo "[${BASH_SOURCE[0]}]: ${option} requires an argument"
Expand Down Expand Up @@ -85,6 +84,8 @@ fi
#------------------------------------
# INCLUDE PARTIAL BUILD
#------------------------------------
# Turn off some shellcheck warnings because we want to have
# variables with multiple arguments.
# shellcheck disable=SC2086,SC2248
source ./partial_build.sh ${_verbose_opt} ${_partial_opt}
# shellcheck disable=
Expand All @@ -97,6 +98,7 @@ fi
#------------------------------------
# Exception Handling Init
#------------------------------------
# Disable shellcheck warning about single quotes not being substituted.
# shellcheck disable=SC2016
ERRSCRIPT=${ERRSCRIPT:-'eval [[ $err = 0 ]]'}
# shellcheck disable=
Expand Down

0 comments on commit 521cf30

Please sign in to comment.