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

Dealing with security fix and windows test.#159

Merged
eddiemoore merged 3 commits intocodecov:masterfrom
SaferNodeJS:master
Mar 9, 2020
Merged

Dealing with security fix and windows test.#159
eddiemoore merged 3 commits intocodecov:masterfrom
SaferNodeJS:master

Conversation

@ralic
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Contributor Author

ralic commented Nov 11, 2019

Anyone here?

@eddiemoore
Copy link
Copy Markdown
Collaborator

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

@ralic
Copy link
Copy Markdown
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