[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks. [WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version. Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this. if it does not -- consider reporting an issue to that repo. [INFO] Initializing environment for https://github.com/shellcheck-py/shellcheck-py. [INFO] Initializing environment for https://github.com/psf/black-pre-commit-mirror. [INFO] Initializing environment for https://github.com/ansible/ansible-lint. [INFO] Initializing environment for https://github.com/ansible/ansible-lint:netaddr,jmespath. [INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/shellcheck-py/shellcheck-py. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/psf/black-pre-commit-mirror. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/ansible/ansible-lint. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... fix end of files.........................................................Passed trim trailing whitespace.................................................Passed mixed line ending........................................................Passed check BOM - deprecated: use fix-byte-order-marker........................Passed check that executables have shebangs.....................................Passed check for merge conflicts................................................Passed check for broken symlinks................................................Passed debug statements (python)................................................Passed shellcheck...............................................................Passed - hook id: shellcheck - duration: 0.74s In docs/source/build-docs.sh line 9: python -m venv ${VENV_DIR} && source ${VENV_DIR}/bin/activate ^----------------------^ SC1091 (info): Not following: ./bin/activate was not specified as input (see shellcheck -x). In roles/devscripts/files/lv-cinder-volumes.sh line 13: pvcreate ${disk} ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: pvcreate "${disk}" In roles/devscripts/files/lv-cinder-volumes.sh line 17: vgcreate cinder-volumes ${disk_str} ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: vgcreate cinder-volumes "${disk_str}" In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 26: source /etc/nat64/config-data ^--------------------^ SC1091 (info): Not following: /etc/nat64/config-data was not specified as input (see shellcheck -x). In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 45: sed -i s~%%_NAT64_HOST_IPV6_%%~${NAT64_HOST_IPV6%%/*}~g dnsmasq.conf ^--------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_HOST_IPV6_%%~"${NAT64_HOST_IPV6%%/*}"~g dnsmasq.conf In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 49: sed -i s~%%_NAT64_IPV6_PREFIX_%%~${NAT64_IPV6_PREFIX}~g unbound.conf ^------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_IPV6_PREFIX_%%~"${NAT64_IPV6_PREFIX}"~g unbound.conf In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 50: sed -i s~%%_NAT64_TAYGA_IPV6_PREFIX_%%~${NAT64_TAYGA_IPV6_PREFIX}~g unbound.conf ^------------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_TAYGA_IPV6_PREFIX_%%~"${NAT64_TAYGA_IPV6_PREFIX}"~g unbound.conf In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 54: sed -i s~%%_NAT64_TAYGA_IPV4_%%~${NAT64_TAYGA_IPV4}~g default.conf ^-----------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_TAYGA_IPV4_%%~"${NAT64_TAYGA_IPV4}"~g default.conf In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 55: sed -i s~%%_NAT64_TAYGA_IPV6_PREFIX_%%~${NAT64_TAYGA_IPV6_PREFIX}~g default.conf ^------------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_TAYGA_IPV6_PREFIX_%%~"${NAT64_TAYGA_IPV6_PREFIX}"~g default.conf In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 56: sed -i s~%%_NAT64_TAYGA_DYNAMIC_POOL_%%~${NAT64_TAYGA_DYNAMIC_POOL}~g default.conf ^-------------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_TAYGA_DYNAMIC_POOL_%%~"${NAT64_TAYGA_DYNAMIC_POOL}"~g default.conf In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 60: sed -i s~%%_NAT64_TAYGA_IPV6_%%~${NAT64_TAYGA_IPV6}~g nat64.nmconnection ^-----------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_TAYGA_IPV6_%%~"${NAT64_TAYGA_IPV6}"~g nat64.nmconnection In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 61: sed -i s~%%_NAT64_TAYGA_IPV6_PREFIX_%%~${NAT64_TAYGA_IPV6_PREFIX}~g nat64.nmconnection ^------------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_TAYGA_IPV6_PREFIX_%%~"${NAT64_TAYGA_IPV6_PREFIX}"~g nat64.nmconnection In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 62: sed -i s~%%_NAT64_TAYGA_DYNAMIC_POOL_%%~${NAT64_TAYGA_DYNAMIC_POOL}~g nat64.nmconnection ^-------------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_TAYGA_DYNAMIC_POOL_%%~"${NAT64_TAYGA_DYNAMIC_POOL}"~g nat64.nmconnection In roles/nat64_appliance/files/elements/nat64-router/static/usr/local/sbin/configure-nat64 line 66: sed -i s~%%_NAT64_TAYGA_DYNAMIC_POOL_%%~${NAT64_TAYGA_DYNAMIC_POOL}~g nat64.nft ^-------------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sed -i s~%%_NAT64_TAYGA_DYNAMIC_POOL_%%~"${NAT64_TAYGA_DYNAMIC_POOL}"~g nat64.nft In roles/openshift_provisioner_node/files/add_bridge_port.sh line 13: CONN_NAME=$(nmcli -t -f GENERAL.CONNECTION dev show ${IFACE_NAME} | cut -d ':' -f 2) ^-----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: CONN_NAME=$(nmcli -t -f GENERAL.CONNECTION dev show "${IFACE_NAME}" | cut -d ':' -f 2) In roles/openshift_provisioner_node/files/add_bridge_port.sh line 16: check_port=$(nmcli con show | grep -c ${PORT_NAME}) || true ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: check_port=$(nmcli con show | grep -c "${PORT_NAME}") || true In roles/openshift_provisioner_node/files/add_bridge_port.sh line 18: if [ ${check_port} -ne 0 ]; then ^-----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: if [ "${check_port}" -ne 0 ]; then In roles/openshift_provisioner_node/files/add_bridge_port.sh line 23: check_iface=$(nmcli dev status | grep -c ${IFACE_NAME}) || true ^-----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: check_iface=$(nmcli dev status | grep -c "${IFACE_NAME}") || true In roles/openshift_provisioner_node/files/add_bridge_port.sh line 25: if [ ${check_iface} -eq 0 ]; then ^------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: if [ "${check_iface}" -eq 0 ]; then In roles/update/files/continuous-test.sh line 57: FILE=$(basename $0) ^-- SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: FILE=$(basename "$0") In roles/update/files/continuous-test.sh line 75: kill -s USR1 $CT_PID ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: kill -s USR1 "$CT_PID" In roles/update/files/continuous-test.sh line 89: l|+l) ^-- SC2214 (warning): This case is not specified by getopts. In roles/update/files/continuous-test.sh line 92: p|+p) ^-- SC2214 (warning): This case is not specified by getopts. In roles/update/files/continuous-test.sh line 95: o|+o) ^-- SC2214 (warning): This case is not specified by getopts. In roles/update/files/continuous-test.sh line 98: d|+d) ^-- SC2214 (warning): This case is not specified by getopts. In roles/update/files/continuous-test.sh line 113: export CT_TTY=$(tty) ^----^ SC2155 (warning): Declare and assign separately to avoid masking return values. In roles/update/files/continuous-test.sh line 117: exec 2>$CT_TTY ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: exec 2>"$CT_TTY" In roles/update/files/continuous-test.sh line 120: export CT_SCRIPT="${@:?'SCRIPT cannot be empty.'}" ^-- SC2124 (warning): Assigning an array to a string! Assign as array, or use * instead of @ to concatenate. In roles/update/files/continuous-test.sh line 125: setsid ${CT_DIR}/${FILE} "$@" $CT_TTY 2>$CT_TTY & ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: setsid "${CT_DIR}"/"${FILE}" "$@" "$CT_TTY" 2>"$CT_TTY" & In roles/update/files/continuous-test.sh line 140: exec 2> ${CT_TTY} ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: exec 2> "${CT_TTY}" In roles/update/files/continuous-test.sh line 141: exec 1> ${CT_TTY} ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: exec 1> "${CT_TTY}" In roles/update/files/continuous-test.sh line 163: setsid ${CT_DIR}/$FILE "$@" $CT_TTY ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: setsid "${CT_DIR}"/"$FILE" "$@" "$CT_TTY" In roles/update/files/continuous-test.sh line 171: date > $CT_ENDFILE ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: date > "$CT_ENDFILE" In roles/update/files/continuous-test.sh line 176: exec >>$CT_LOGFILE ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: exec >>"$CT_LOGFILE" In roles/update/files/continuous-test.sh line 183: start_time_h="$(date -d@${start_time})" ^-----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: start_time_h="$(date -d@"${start_time}")" In roles/update/files/continuous-test.sh line 186: "${CT_SCRIPT}" ${CT_SCRIPT_ARGS} &>> "${CT_CMD_OUT_DIR}/${start_time}.log" ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "${CT_SCRIPT}" "${CT_SCRIPT_ARGS}" &>> "${CT_CMD_OUT_DIR}/${start_time}.log" In scripts/bindep-install line 26: export BINDEP_FILE="${BINDEP_FILE:-$(dirname $(readlink -f ${BASH_SOURCE[0]}))/../bindep.txt}" ^-- SC2046 (warning): Quote this to prevent word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: export BINDEP_FILE="${BINDEP_FILE:-$(dirname $(readlink -f "${BASH_SOURCE[0]}"))/../bindep.txt}" In scripts/bindep-install line 32: source /etc/os-release || source /usr/lib/os-release ^-------------^ SC1091 (info): Not following: /etc/os-release was not specified as input (see shellcheck -x). ^-----------------^ SC1091 (info): Not following: /usr/lib/os-release was not specified as input (see shellcheck -x). In scripts/bindep-install line 40: BINDEP_PKGS=$(${HOME}/test-python/bin/bindep -b -f "${BINDEP_FILE}" test || true) ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: BINDEP_PKGS=$("${HOME}"/test-python/bin/bindep -b -f "${BINDEP_FILE}" test || true) In scripts/bindep-install line 47: if [[ ${#BINDEP_PKGS} > 0 ]]; then ^-- SC2071 (error): > is for string comparisons. Use -gt instead. In scripts/bindep-install line 50: sudo "${RHT_PKG_MGR}" install -y ${BINDEP_PKGS} ^------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sudo "${RHT_PKG_MGR}" install -y "${BINDEP_PKGS}" In scripts/check_k8s_snippets_comment.sh line 2: set -o pipefail ^------^ SC3040 (warning): In POSIX sh, set option pipefail is undefined. In scripts/check_k8s_snippets_comment.sh line 7: if [[ $missing_comment != '' ]]; then ^--------------------------^ SC3010 (warning): In POSIX sh, [[ ]] is undefined. In scripts/check_k8s_snippets_comment.sh line 13: let "exit_code+=1" ^---------------^ SC3039 (warning): In POSIX sh, 'let' is undefined. In scripts/check_k8s_snippets_comment.sh line 17: while read match; do ^--^ SC2162 (info): read without -r will mangle backslashes. In scripts/check_k8s_snippets_comment.sh line 18: tmpl=$(echo -n "${match}" | cut -d ':' -f 1) ^-- SC3037 (warning): In POSIX sh, echo flags are undefined. In scripts/check_k8s_snippets_comment.sh line 19: comment=$(echo -n "${match}" | cut -d ':' -f 3 | tr -d '[:space:]') ^-- SC3037 (warning): In POSIX sh, echo flags are undefined. In scripts/check_k8s_snippets_comment.sh line 20: if [[ "${tmpl}" != "${comment}" ]]; then ^-----------------------------^ SC3010 (warning): In POSIX sh, [[ ]] is undefined. In scripts/check_k8s_snippets_comment.sh line 21: let "exit_code+=1" ^---------------^ SC3039 (warning): In POSIX sh, 'let' is undefined. In scripts/check_k8s_snippets_comment.sh line 24: done <<< ${set_path} ^-^ SC3011 (warning): In POSIX sh, here-strings are undefined. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: done <<< "${set_path}" In scripts/prepare_doc_build_env.sh line 4: PROJECT_DIR="$(dirname $(readlink -f $0))/../" ^---------------^ SC2046 (warning): Quote this to prevent word splitting. ^-- SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: PROJECT_DIR="$(dirname $(readlink -f "$0"))/../" In scripts/prepare_doc_build_env.sh line 15: for i in ${PROJECT_DIR}/roles/*/README.md; do ^------------^ SC2231 (info): Quote expansions in this for loop glob to prevent wordsplitting, e.g. "$dir"/*.txt . In scripts/prepare_doc_build_env.sh line 16: dir_name=$(dirname ${i}) ^--^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: dir_name=$(dirname "${i}") In scripts/prepare_doc_build_env.sh line 17: role_name=$(basename ${dir_name}) ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: role_name=$(basename "${dir_name}") In scripts/prepare_doc_build_env.sh line 18: test -L docs/source/roles/${role_name}.md || \ ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: test -L docs/source/roles/"${role_name}".md || \ In scripts/prepare_doc_build_env.sh line 19: ln -s ../../../roles/${role_name}/README.md \ ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ln -s ../../../roles/"${role_name}"/README.md \ In scripts/prepare_doc_build_env.sh line 20: docs/source/roles/${role_name}.md ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: docs/source/roles/"${role_name}".md In scripts/run_ansible_test line 24: PROJECT_DIR="$(dirname $(readlink -f ${BASH_SOURCE[0]}))/../" ^-- SC2046 (warning): Quote this to prevent word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: PROJECT_DIR="$(dirname $(readlink -f "${BASH_SOURCE[0]}"))/../" In scripts/run_ansible_test line 38: collection_path="${HOME}/.ansible/collections/ansible_collections" ^-------------^ SC2034 (warning): collection_path appears unused. Verify use (or export if used externally). In scripts/run_ansible_test line 54: pushd ${HOME}/.ansible/collections/ansible_collections/cifmw/general ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: pushd "${HOME}"/.ansible/collections/ansible_collections/cifmw/general In scripts/run_molecule line 24: PROJECT_DIR="$(dirname $(readlink -f ${BASH_SOURCE[0]}))/../" ^-- SC2046 (warning): Quote this to prevent word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: PROJECT_DIR="$(dirname $(readlink -f "${BASH_SOURCE[0]}"))/../" In scripts/run_molecule line 34: local_roledir=$(echo $ROLE_DIR | sed 's|\.\?\./||g') ^-- SC2001 (style): See if you can use ${variable//search/replace} instead. ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local_roledir=$(echo "$ROLE_DIR" | sed 's|\.\?\./||g') In scripts/run_molecule line 52: role=$(echo $rolepath|sed "s|${local_roledir}||" | cut -f1 -d /) ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: role=$(echo "$rolepath"|sed "s|${local_roledir}||" | cut -f1 -d /) In scripts/run_molecule line 53: test -d ${role} || continue ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: test -d "${role}" || continue In scripts/run_molecule line 55: pushd $role ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: pushd "$role" In scripts/run_molecule line 58: ${HOME}/test-python/bin/molecule -c "${PROJECT_DIR}${MOLECULE_CONFIG}" ${TEST_VERBOSITY} test --all ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "${HOME}"/test-python/bin/molecule -c "${PROJECT_DIR}${MOLECULE_CONFIG}" "${TEST_VERBOSITY}" test --all In scripts/run_molecule line 61: molecule -c "${PROJECT_DIR}${MOLECULE_CONFIG}" ${TEST_VERBOSITY} test --all ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: molecule -c "${PROJECT_DIR}${MOLECULE_CONFIG}" "${TEST_VERBOSITY}" test --all In scripts/setup_env line 23: export PROJECT_DIR="$(dirname $(dirname $(readlink -f ${BASH_SOURCE[0]})))" ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-- SC2046 (warning): Quote this to prevent word splitting. ^-- SC2046 (warning): Quote this to prevent word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: export PROJECT_DIR="$(dirname $(dirname $(readlink -f "${BASH_SOURCE[0]}")))" In scripts/setup_env line 32: source /etc/os-release || source /usr/lib/os-release ^-------------^ SC1091 (info): Not following: /etc/os-release was not specified as input (see shellcheck -x). ^-----------------^ SC1091 (info): Not following: /usr/lib/os-release was not specified as input (see shellcheck -x). In scripts/setup_env line 33: RHT_PKG_MGR=$(command -v dnf) ^---------^ SC2034 (warning): RHT_PKG_MGR appears unused. Verify use (or export if used externally). In scripts/setup_env line 66: mkdir -p ${HOME}/ci/yum.repos.d ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: mkdir -p "${HOME}"/ci/yum.repos.d In scripts/setup_env line 67: cp /etc/ci/mirror_info.sh ${HOME}/ci || touch ${HOME}/ci/mirror_info.sh ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: cp /etc/ci/mirror_info.sh "${HOME}"/ci || touch "${HOME}"/ci/mirror_info.sh In scripts/setup_env line 68: cp -r /opt/yum.repos.d/* ${HOME}/ci/yum.repos.d || cp -r /etc/yum.repos.d/* ${HOME}/ci/yum.repos.d ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: cp -r /opt/yum.repos.d/* "${HOME}"/ci/yum.repos.d || cp -r /etc/yum.repos.d/* "${HOME}"/ci/yum.repos.d In scripts/setup_env line 85: ${SYSTEM_PIP} install ${PIP_INSTALL_ARGUMENTS} ^----------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ${SYSTEM_PIP} install "${PIP_INSTALL_ARGUMENTS}" In scripts/setup_molecule line 23: export PROJECT_DIR="$(dirname $(dirname $(readlink -f ${BASH_SOURCE[0]})))" ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-- SC2046 (warning): Quote this to prevent word splitting. ^-- SC2046 (warning): Quote this to prevent word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: export PROJECT_DIR="$(dirname $(dirname $(readlink -f "${BASH_SOURCE[0]}")))" In scripts/setup_molecule line 45: ${PIP} install ${PIP_INSTALL_ARGUMENTS} ^----------------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ${PIP} install "${PIP_INSTALL_ARGUMENTS}" In scripts/setup_molecule line 51: ${GALAXY} collection install --upgrade --force --timeout=120 ${PROJECT_DIR} ^------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ${GALAXY} collection install --upgrade --force --timeout=120 "${PROJECT_DIR}" For more information: https://www.shellcheck.net/wiki/SC2071 -- > is for string comparisons. Use ... https://www.shellcheck.net/wiki/SC2034 -- RHT_PKG_MGR appears unused. Verif... https://www.shellcheck.net/wiki/SC2046 -- Quote this to prevent word splitt... black....................................................................Failed - hook id: black - files were modified by this hook reformatted scripts/create_role_molecule.py All done! ✨ 🍰 ✨ 1 file reformatted, 54 files left unchanged. Ansible-lint.............................................................Passed