Move promise.spread to promise.join for clarity (from review)

This commit is contained in:
Tim Perry 2017-06-15 12:34:26 +02:00
parent cf7bf2cb7d
commit a29b40eefa
2 changed files with 2 additions and 3 deletions

View File

@ -279,7 +279,7 @@ exports.bufferImage = function(docker, imageId, bufferFile) {
streamUtils = require('./streams');
image = docker.getImage(imageId);
imageMetadata = image.inspectAsync();
return Promise.all([image.get(), imageMetadata.get('Size')]).spread(function(imageStream, imageSize) {
return Promise.join(image.get(), imageMetadata.get('Size'), function(imageStream, imageSize) {
return streamUtils.buffer(imageStream, bufferFile).tap(function(bufferedStream) {
return bufferedStream.length = imageSize;
});

View File

@ -290,8 +290,7 @@ exports.bufferImage = (docker, imageId, bufferFile) ->
image = docker.getImage(imageId)
imageMetadata = image.inspectAsync()
Promise.all([image.get(), imageMetadata.get('Size')])
.spread (imageStream, imageSize) ->
Promise.join image.get(), imageMetadata.get('Size'), (imageStream, imageSize) ->
streamUtils.buffer(imageStream, bufferFile)
.tap (bufferedStream) ->
bufferedStream.length = imageSize