Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(deps): remove high severity vulnerability and add extra travis check for PRs #243

Merged

Conversation

erickwilder
Copy link
Contributor

The only exception is webpack-svgstore-plugin that still points to some older package versions
with low severity vulnerabilities and apparently, the repo is not active that the actual fix
could be merged. Because of that, I'm limiting the scope to just high severity issues.

Note: I decided to include audit-ci because yarn does not provide (yet) a way to filter
violations by severity level.

@erickwilder erickwilder force-pushed the fix-audit-vulnerabilities branch from 97e7d94 to 4e0edc9 Compare March 20, 2019 00:17
@erickwilder
Copy link
Contributor Author

@hanspagel / @philippkuehn

If you have some time later to review this one it would be awesome.

…eck on PRS

The only exception is `webpack-svgstore-plugin` that still points to some older package versions
with `low` severity vulnerabilities and apparently the repo is not active that [the actual fix](mrsum/webpack-svgstore-plugin#172)
could be merged. Because of that I'm limiting the scope to just `high` severity issues.

Note: I decided to include `audit-ci` because `yarn` does not provide (yet) a way to filter
violations by severity level.

fix(deps): upgrade packages so that `yarn lint` works with the current babel-eslint
@erickwilder erickwilder force-pushed the fix-audit-vulnerabilities branch from 5a8d3ce to 18ba26d Compare March 31, 2019 08:52
@philippkuehn philippkuehn merged commit 3e1f187 into ueberdosis:master Apr 11, 2019
@philippkuehn
Copy link
Contributor

Thanks, Erick! This is great!

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.

2 participants