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

Commit 244289e

Browse files
[PR Feedback] args -> secret_args for specificity and clarity
Signed-off-by: Christopher Maier <[email protected]>
1 parent 436dc41 commit 244289e

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

hooks/environment

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,10 +85,10 @@ fi
8585
readonly container_name="vault-env-plugin-${BUILDKITE_JOB_ID}"
8686

8787
envconsul_env() {
88-
args=()
88+
secret_args=()
8989
for secret in "${secrets[@]}"; do
9090
# secret_prefix is guaranteed to end with a / if it is non-empty
91-
args+=("-secret=${secret_prefix}${secret}")
91+
secret_args+=("-secret=${secret_prefix}${secret}")
9292
done
9393

9494
# Explicitly *not* using `--rm` so we can output the container
@@ -98,7 +98,7 @@ envconsul_env() {
9898
--name="${container_name}" \
9999
-- \
100100
"${image}" \
101-
"${args[@]}" \
101+
"${secret_args[@]}" \
102102
-once \
103103
-upcase \
104104
-pristine \

0 commit comments

Comments
 (0)