search

AgustinCB / docker-api

Docker Remote API driver for node.js. It uses the same modem than dockerode, but the interface is promisified and with a fancier syntax.
GNU General Public License v3.0
303 stars 51 forks source link

upgrade all dependencies #43

Closed dcharbonnier closed 3 years ago

dcharbonnier commented 6 years ago

If you get why I had to change the test for exec it should be safe to merge

const stream = await exec.start({stream: true })

without the stream parameter the test fail

dcharbonnier commented 6 years ago

so there is an other problem ;-)

dcharbonnier commented 6 years ago

can you retry the build @AgustinCB ? I don't think it's related to the change, like sometimes you get a 

  1. container › kill
  AssertionError:   
    "removing"
        Test.<anonymous> (test/container.js:153:5)