Skip to content

Commit

Permalink
Fix: --aggregate-output cannot handle large data (fixes #111)(#112)
Browse files Browse the repository at this point in the history
  • Loading branch information
jlopezxs authored and mysticatea committed Nov 3, 2017
1 parent 6ef64ab commit 096779b
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions lib/run-tasks.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
// Requirements
//------------------------------------------------------------------------------

const streams = require("memory-streams")
const MemoryStream = require("memorystream")
const NpmRunAllError = require("./npm-run-all-error")
const runTask = require("./run-task")

Expand Down Expand Up @@ -110,7 +110,9 @@ module.exports = function runTasks(tasks, options) {

const originalOutputStream = options.stdout
const optionsClone = Object.assign({}, options)
const writer = new streams.WritableStream()
const writer = new MemoryStream(null, {
readable: false,
})

if (options.aggregateOutput) {
optionsClone.stdout = writer
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
"ansi-styles": "^3.2.0",
"chalk": "^2.1.0",
"cross-spawn": "^5.1.0",
"memory-streams": "^0.1.2",
"memorystream": "^0.3.1",
"minimatch": "^3.0.4",
"ps-tree": "^1.1.0",
"read-pkg": "^2.0.0",
Expand Down

0 comments on commit 096779b

Please sign in to comment.