From 0dbcc8b22403779f6523fc71b944b6ad0a24c62c Mon Sep 17 00:00:00 2001 From: "Tobin C. Harding" Date: Wed, 21 Aug 2024 16:35:02 +1000 Subject: [PATCH] Use say_err We have an error function, use it. This clears a shellcheck warning (unseen locally by myself) and puts the error message below the ouput from `cargo tree` which is probably better if its long. --- ci/run_task.sh | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ci/run_task.sh b/ci/run_task.sh index 05b5738..b3a0abd 100755 --- a/ci/run_task.sh +++ b/ci/run_task.sh @@ -256,9 +256,8 @@ do_dup_deps() { duplicate_dependencies=$(eval "$tree_cmd") if [ "$duplicate_dependencies" -ne 0 ]; then - echo "Dependency tree is broken, contains duplicates" cargo tree --target=all --all-features --duplicates - exit 1 + say_err "Dependency tree is broken, contains duplicates" fi set -o pipefail