Skip to content

Commit 838f4a6

Browse files
authored
Merge pull request #2195 from o1-labs/dw/shellcheck
Scripts: make shellcheck happy on download-bindings.sh
2 parents fbfeb58 + 5e1f669 commit 838f4a6

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/bindings/scripts/download-bindings.sh

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
1-
# /usr/bin/env bash
1+
#!/usr/bin/env bash
2+
23
set -e
34

45
(which gh > /dev/null 2>&1) || (echo "Please install gh the github cli tool: https://github.com/cli/cli#installation" && exit 1)
56

67
REV=${REV:=$(git rev-parse HEAD)}
78
RUN_ID=$( \
8-
gh run list --commit $REV --json name,databaseId | \
9+
gh run list --commit "${REV}" --json name,databaseId | \
910
jq -r '.[] | select(.name == "Checks" or .name == "Build and upload bindings") | .databaseId' \
1011
)
1112

@@ -21,7 +22,7 @@ then
2122
fi
2223

2324
gh run watch "$RUN_ID" --exit-status || echo "Warning: ci failed on this job, trying to download bindings anyway"
24-
#remove if it exists but don't fail
25+
# remove if it exists but don't fail
2526
if [ -f ".bindings_download/bindings.tar.gz" ]; then
2627
rm .bindings_download/bindings.tar.gz
2728
fi

0 commit comments

Comments
 (0)