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

Dealing with security fix and windows test. #159

Merged
merged 3 commits into from
Mar 9, 2020

Conversation

ralic
Copy link
Contributor

@ralic ralic commented Oct 24, 2019

Before

found 15492 vulnerabilities (15491 high, 1 critical)
  run `npm audit fix` to fix them, or `npm audit` for details

After Fix

found 0 vulnerabilities

Test passed :
https://travis-ci.org/SaferNodeJS/codecov-node/builds/602090361

@codecov
Copy link

codecov bot commented Oct 24, 2019

Codecov Report

Merging #159 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #159   +/-   ##
=======================================
  Coverage   90.14%   90.14%           
=======================================
  Files          23       23           
  Lines         355      355           
  Branches       85       85           
=======================================
  Hits          320      320           
  Misses         35       35           

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 bac0787...e138efe. Read the comment docs.

@ralic ralic force-pushed the master branch 2 times, most recently from 570d963 to 76e82d3 Compare October 24, 2019 00:17
@ralic
Copy link
Contributor Author

ralic commented Nov 11, 2019

Anyone here?

@eddiemoore
Copy link
Collaborator

Sorry it's taken so long to get to this. Please update the merge conflicts and we can recheck

@ralic
Copy link
Contributor Author

ralic commented Feb 4, 2020

Done.

@eddiemoore eddiemoore merged commit 55d69cd into codecov:master Mar 9, 2020
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.

2 participants