Attempt at fixing CI issue on windows

[heyimalex]

Does CI run tests right from the source? Let's find out!

[iansu]

It looks like this did the trick? cc @ianschmitz

[heyimalex]

I feel pretty confident that removing the kill call won't negatively affect anything. It's pretty much the very last thing done before the job completes, and at that point all orphaned processed get killed anyway. Maybe if we spawned verdaccio again after there would be a port conflict, but we don't.

We could set +e before and set -e after, but really that's just ignoring the failure on some platforms. We could track the pid of the spawned process and then use that to kill it, but that's adding complexity for little benefit. I think just removing it and dealing with it later if anything breaks is the best way forward.

tldr: merge away! I'm gonna rewrite the commit message right now.

[iansu]

Works for me. Thanks!