Skip to content
This repository was archived by the owner on Dec 8, 2021. It is now read-only.

Commit 2933c22

Browse files
authored
fix: avoid/quash some shellcheck warnings (#148)
SC1091: Disable if we don't statically know the source path. SC2016: `shellcheck` thinks the `` `/\\` `` is a command substitution. SC2086: Double quote ${BINDIR} expansion.
1 parent 558ff88 commit 2933c22

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed

ci/templates/generate-kokoro-install.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,11 @@ BUILD_NAMES=(
4949
)
5050
readonly BUILD_NAMES
5151

52-
# shellcheck source=../../ci/etc/kokoro/docker-fragments.sh
52+
# shellcheck disable=SC1091
5353
source "${PROJECT_ROOT}/ci/templates/kokoro/docker-fragments-functions.sh"
54-
# shellcheck source=../ci/../etc/kokoro/docker-fragments.sh
54+
# shellcheck disable=SC1091
5555
source "${PROJECT_ROOT}/ci/templates/kokoro/docker-fragments.sh"
56-
# shellcheck source=../../ci/etc/kokoro/install/project-config.sh
56+
# shellcheck disable=SC1091
5757
source "${DESTINATION_ROOT}/ci/etc/kokoro/install/project-config.sh"
5858

5959
generate_dockerfile() {

ci/templates/generate-kokoro-readme.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,11 @@ BUILD_NAMES=(
4949
)
5050
readonly BUILD_NAMES
5151

52-
# shellcheck source=../../ci/etc/kokoro/docker-fragments.sh
52+
# shellcheck disable=SC1091
5353
source "${PROJECT_ROOT}/ci/templates/kokoro/docker-fragments-functions.sh"
54-
# shellcheck source=../../ci/etc/kokoro/docker-fragments.sh
54+
# shellcheck disable=SC1091
5555
source "${PROJECT_ROOT}/ci/templates/kokoro/docker-fragments.sh"
56-
# shellcheck source=../../ci/etc/kokoro/readme/project-config.sh
56+
# shellcheck disable=SC1091
5757
source "${DESTINATION_ROOT}/ci/etc/kokoro/readme/project-config.sh"
5858

5959
generate_dockerfile() {

ci/templates/kokoro/docker-fragments-functions.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ replace_fragments() {
2828

2929
local sed_args=()
3030
for fragment in "${fragment_names[@]}"; do
31+
# shellcheck disable=SC2016
3132
sed_args+=("-e" "s,@${fragment}@,$(/bin/echo -n "${!fragment}" |
3233
# Note the use of \003 ("End of Text") as the magic character to
3334
# represent newlines. This must match the invert `tr` call below. It

ci/test-readme/generate-readme.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ Google Cloud C++ Client Libraries.
4141
4242
_EOF_
4343

44-
${BINDIR}/../generate-badges.sh
44+
"${BINDIR}/../generate-badges.sh"
4545

4646
cat <<'_EOF_'
4747
## Table of Contents

0 commit comments

Comments
 (0)