Skip to content

Conversation

@gbhasha
Copy link

@gbhasha gbhasha commented Feb 14, 2020

and running 'npm audit fix' updated the other npm dependencies to fix the vulnerabilities.

This PR fixes #75

@codecov
Copy link

codecov bot commented Feb 14, 2020

Codecov Report

Merging #91 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #91   +/-   ##
=======================================
  Coverage   96.47%   96.47%           
=======================================
  Files           2        2           
  Lines         142      142           
  Branches       39       39           
=======================================
  Hits          137      137           
  Misses          5        5

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 3491874...834094e. Read the comment docs.

@gbhasha
Copy link
Author

gbhasha commented Feb 16, 2020

@40818419 can you please look into this pull request? faster response is much appreciated...

@sirajalam049
Copy link
Contributor

Why in dev-dependencies, they must be in the peer dependencies.

@andrewl913
Copy link

should put them in both dev and peer dependencies so consumers will be aware.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

package.json misconfigured

3 participants