Setup Appveyor

[gtramontina]

As suggested by @ta2edchimp on #114 (comment)

---

Interestingly enough, a few tests fail running on windows. Three of them are / vs. \ on assertions… (classic! 😛) The fourth and last one is related to setting the PATH… I'll look into those tomorrow.

The last failed test still persists. I'll need a bit more time to investigate as maybe I'll require a VM do help out (or, in the meanwhile, someone could 👀 spot what the issue might be 😉).

I've also made continuous-integration/travis-ci/pr required for PRs to be merged. And as soon as this one gets in (I'm planning on fixing the tests on this same PR), I'll make continuous-integration/appveyor/pr also required… Does this seem reasonable?

[codecov-io]

Current coverage is 100%

Merging #117 into master will not change coverage

@@           master   #117   diff @@
====================================
  Files           4      4          
  Lines         118    118          
  Methods         0      0          
  Messages        0      0          
  Branches        4      4          
====================================
  Hits          118    118          
  Misses          0      0          
  Partials        0      0          

Powered by Codecov. Last updated by e3e552a...bcfea36

[ta2edchimp]

Hmm... The path manipulations within the tests aren't os/path.sep agnostic yet 😞

[gtramontina]

Updated the PR description:

The last failed test still persists. I'll need a bit more time to investigate as maybe I'll require a VM do help out (or, in the meanwhile, someone could 👀 spot what the issue might be 😉).

Reason:

1) runner should alter the path:
     AssertionError: expected undefined to start with C:\projects\ghooks\node_modules\.bin

[ta2edchimp]

Browsed a bit on the tablet. It's calledOptions.env[getPathVar()] resolving to undefined.
After a quick lookup for manage-path's get-path-var method, it returns "PATH".
I guess we should pass process.env and process.platform as arguments to getPathVar.

Maybe @kentcdodds can approve this assumption?

[ta2edchimp]

On a side note, I don't know ... shouldn't the platform be win64 or sth. like that on modern Windows?

[gtramontina]

We can also setup Appveyor to run on the different architectures. Although simply running on the different versions of node already takes forever!

Feel free to take over this branch… I'm going 💤 now… pretty late here. 😄
I'll continue on it tomorrow, anyway…

[kentcdodds]

LGTM. Could we turn off gitcop? It's kind of annoying and heavy-handed...

[gtramontina]

Done! 👍 Gitcop is gone!

[kentcdodds]

FYI, it's been my experience that AppVeyor takes FOREVER.

[gtramontina]

Yeap… It just finished running the tests on the different node versions…
Let's have it like this and later look at how much it is dragging us. It may be the case that we set it up to test against only one version of node…