search

davglass / cpr

Nodejs cp -R
Other
79 stars 80 forks source link

Test on Windows #46

Open stevenvachon opened 7 years ago

stevenvachon commented 7 years ago

…with AppVeyor. If you set up the [free] account, I can upload the config file.

davglass commented 7 years ago

I'm not opposed to adding this, I'm not a Windows user so I would be able to even fix issues if the build were to break.

davglass commented 7 years ago

Added the AppVeyor build info..

davglass commented 7 years ago

It's building now & the build fails (which we knew) but it's ready to be tested on.

stevenvachon commented 7 years ago

I didn't know that it failed on Windows. Why are we using "jenkins-mocha"? I'd think that regular mocha would be sufficient.

Also, the AppVeyor config file could benefit from additional options: https://github.com/stevenvachon/edit-dotenv/blob/master/.appveyor.yml

davglass commented 7 years ago

jenkins-mocha wraps mocha which istanbul and provides coverage in a single command. Not sure why it's not working as I'm not a Windows user. Feel free to send a PR with the proper configs, I've never used Appveyor before and have no idea how it works and little time to explore all the options.

stevenvachon commented 7 years ago

nyc (the all-in-one istanbul CLI) wraps mocha simply:

nyc mocha test.js