Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Removing execSync #45

Merged
merged 1 commit into from
Mar 3, 2017
Merged

Removing execSync #45

merged 1 commit into from
Mar 3, 2017

Conversation

eddiemoore
Copy link
Collaborator

@eddiemoore eddiemoore commented Feb 16, 2017

Removing execSync and making codecov-node require node version 0.12+. This SHOULD make codecov work on node v7

Removed node v0.10 and v0.11 from travis.yml and added v7

Updated version number to 2.0.0 since it will no longer work on node v0.10

Had to comment out one test for now as i'm not quite sure what it is meant to be doing, however it always fails.

@codecov
Copy link

codecov bot commented Feb 16, 2017

Codecov Report

Merging #45 into master will increase coverage by 4.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
+ Coverage   91.41%   95.47%   +4.05%     
==========================================
  Files          17       17              
  Lines         233      221      -12     
  Branches       44       41       -3     
==========================================
- Hits          213      211       -2     
+ Misses         20       10      -10
Impacted Files Coverage Δ
lib/services/localGit.js 100% <ø> (+30.76%)
lib/services/drone.js 100% <ø> (+27.27%)
lib/codecov.js 92.37% <ø> (-0.26%)
lib/detect.js 92.85% <ø> (+21.42%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2408ca9...a8f07b8. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant