Skip to content

Commit 8dee189

Browse files
committed
Merge #689: ci: shellcheck checks
926bb05 ci: shellcheck checks (Jose Storopoli) Pull request description: Following rust-bitcoin/rust-bitcoin#2762, adding CI shellcheck cheks here as well. I also did all fixes that I could find with ```bash shellcheck **/*.sh ``` If I've missed any please let me know. ACKs for top commit: apoelstra: ACK 926bb05 tcharding: ACK 926bb05 Tree-SHA512: 1eb32b6d1f3008a03b70995c3b936921bad9a2ea7f329133e704fd9b4eb9bd7ec5cac7cd5e4a4a2ceff83eec03c785478cb1de37922b5b9d7787c907040876d4
2 parents 6895a06 + 926bb05 commit 8dee189

File tree

5 files changed

+24
-3
lines changed

5 files changed

+24
-3
lines changed

.github/workflows/shellcheck.yml

+16
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
2+
name: Shellcheck
3+
on:
4+
pull_request:
5+
branches:
6+
- master
7+
jobs:
8+
shellcheck:
9+
name: Shellcheck
10+
runs-on: ubuntu-latest
11+
steps:
12+
- uses: actions/checkout@v4
13+
- name: Run ShellCheck
14+
uses: ludeeus/[email protected]
15+
env:
16+
SHELLCHECK_OPTS: -x # allow outside sources

embedded/scripts/env.sh

+2
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,4 @@
1+
# we don't want shebangs in env.sh, disable shellcheck warning
2+
# shellcheck disable=SC2148
13
export RUSTFLAGS="-C link-arg=-Tlink.x"
24
export CARGO_TARGET_THUMBV7M_NONE_EABI_RUNNER="qemu-system-arm -cpu cortex-m3 -machine mps2-an385 -nographic -semihosting-config enable=on,target=native -kernel"

fuzz/cycle.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77

88
set -e
99
REPO_DIR=$(git rev-parse --show-toplevel)
10-
# shellcheck source=./fuzz-util.sh
10+
# can't find the file because of the ENV var
11+
# shellcheck source=/dev/null
1112
source "$REPO_DIR/fuzz/fuzz-util.sh"
1213

1314
while :

fuzz/fuzz.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@ set -ex
33

44
REPO_DIR=$(git rev-parse --show-toplevel)
55

6-
# shellcheck source=./fuzz-util.sh
6+
# can't find the file because of the ENV var
7+
# shellcheck source=/dev/null
78
source "$REPO_DIR/fuzz/fuzz-util.sh"
89

910
# Check that input files are correct Windows file names

fuzz/generate-files.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ set -e
44

55
REPO_DIR=$(git rev-parse --show-toplevel)
66

7-
# shellcheck source=./fuzz-util.sh
7+
# can't find the file because of the ENV var
8+
# shellcheck source=/dev/null
89
source "$REPO_DIR/fuzz/fuzz-util.sh"
910

1011
# 1. Generate fuzz/Cargo.toml

0 commit comments

Comments
 (0)