Skip to content

Commit

Permalink
Fix quoting of docker command
Browse files Browse the repository at this point in the history
The version of bash in the build container doesn't like parens `(...)`
(usually meaning "run in a subshell"), so when I removed the
_evaluation_ `$(...)` in 9b56712 it decided not to run at all. On the
other hand, just removing the parens doesn't fix it: it needs double
quoting because it's being passed to docker.
  • Loading branch information
squaremo committed Apr 3, 2020
1 parent c6607e4 commit a78b6ba
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion run-release.sh
Original file line number Diff line number Diff line change
Expand Up @@ -68,4 +68,4 @@ if [ -z "$NPM_TOKEN" ]; then
exit 1
fi
echo '//registry.npmjs.org/:_authToken=${NPM_TOKEN}' > @jkcfg/std/.npmrc
docker_run bash -c '(cd @jkcfg/std && npm publish)'
docker_run bash -c "'cd @jkcfg/std && npm publish'"

0 comments on commit a78b6ba

Please sign in to comment.