chore: add codecov script to appveyor CI#686
Conversation
|
This is a test to see if codecov is additive: will running codecov on both Windows and Unix take the union of the lines covered? This would be very beneficial for us, so that we can cover lines which handle different versions of Node.js and which handle different platforms. If this is successful, then we should see that code coverage improves, particularly for commands with lots of Windows-specific code, such as |
891d598 to
30006dd
Compare
|
Looks like the codecov package relies on the unix |
Invoke codecov on appveyor builds too, so that we can get coverage for Windows-specific lines of code. Partial fix for #671
30006dd to
18b4b82
Compare
|
Ok, I think appveyor works now. The tricks I needed were:
|
Codecov Report
@@ Coverage Diff @@
## master #686 +/- ##
=========================================
+ Coverage 92.68% 94.3% +1.61%
=========================================
Files 33 33
Lines 1176 1176
=========================================
+ Hits 1090 1109 +19
+ Misses 86 67 -19
Continue to review full report at Codecov.
|
Invoke codecov on appveyor builds too, so that we can get coverage for
Windows-specific lines of code.
Partial fix for #671