pretest script fails on windows about d3-random HOT 6 CLOSED

👍 verified in git-bash, win-bash and cygwin

from d3-random.

I do not have a Windows machine, so unfortunately I have no way of reproducing this issue. However, I suspect that the failure is caused by the zip command. This didn’t need to be run on prepublish, so I’ve moved it to postpublish, and hopefully that means npm install now succeeds.

If that’s not the case, please help me out by sending a pull request to fix the issue. I’m afraid I would just be guessing blindly trying to fix it myself!

from d3-random.

It still crashes in the exact same way as before your fix. I'll have a look at it and try to fix it. No promises though. :)

from d3-random.

from d3-random.

Yeah, I think the problem is somewhere in the pretest script actually. I'll let you know if I work it out.

from d3-random.

So I found the problem, which was that windows parsed the embedded quotes inside the pretest script incorrectly, so I changed the script to:

"pretest": "mkdir -p build && node ./scripts/pretest.js > build/version.js && rollup -f umd -n d3_random -o build/d3-random.js -- index.js"

Which sort of worked, however, my terminal window now hangs indefinitely on:

> [email protected] test C:\Users\Pepe\dropbox\workspace\d3-random
> faucet find test -name '*-test.js' && eslint index.js src

Not sure what that is about...

P.S shouldn't this issue be reopened?

from d3-random.