These are chat archives for deployd/contributors

13th
Jan 2015
Andrei Alecu
@andreialecu
Jan 13 2015 14:13
deployd/deployd#480 any chance this could be merged soon-ish? It's innocent enough, only a testing infrastructure thing
and as with the phantomjs runner, it's very important to ensure code correctness, I keep having to merge it and test then revert
Andrei Alecu
@andreialecu
Jan 13 2015 14:37
and by the way: deployd/deployd#476 , another positive side effect of the dpd cli change in this PR is that it tests if the cli works properly and can spawn a mongod instance. previously there were no tests for that - this at least ensures that npm test will fail if there is something wrong somewhere in /bin/dpd
Nicolas Ritouet
@NicolasRitouet
Jan 13 2015 15:35
@andreialecu , I just rebased your PR, thx
Andrei Alecu
@andreialecu
Jan 13 2015 15:36
just saw that, thx
I'm fixing something else I found, just caught it while working on my stuff
expect new PR
should be minor
Nicolas Ritouet
@NicolasRitouet
Jan 13 2015 15:39
ok :sparkles:
Andrei Alecu
@andreialecu
Jan 13 2015 16:32
deployd/deployd#485
is this squashable? I pushed to the same branch I used before instead of making another