Commit e69079de authored by Robert Knight's avatar Robert Knight

Fix command to publish npm package to S3

The previous syntax ended up executing:

  docker run -d hypothesis s3://cdn.hypothes.is nickstenning/s3-npm-publish

So `hypothesis` was interpreted as the Docker image name instead of an
argument. Passing the args as the second ('command') argument to
`Image.withRun` results in the correct command being executed:

  docker run -d nickstenning/s3-npm-publish hypothesis s3://cdn.hypothes.is
parent 00dfa406
...@@ -30,8 +30,8 @@ node { ...@@ -30,8 +30,8 @@ node {
// Upload the contents of the package to an S3 bucket, which it // Upload the contents of the package to an S3 bucket, which it
// will then be served from. // will then be served from.
s3publish = docker.image('nickstenning/s3-npm-publish') docker.image('nickstenning/s3-npm-publish')
s3publish.withRun('hypothesis s3://cdn.hypothes.is') { /* empty */ } .withRun('', 'hypothesis s3://cdn.hypothes.is') { /* empty */ }
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment