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.

Sign up for GitHub

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

Jump to bottom

Add audit script to check dependency vulnerabilities #1020

Merged
merged 4 commits into from
Feb 5, 2019
Merged

Add audit script to check dependency vulnerabilities #1020

merged 4 commits into from
Feb 5, 2019

Conversation

Gnito
Copy link
Contributor

@Gnito Gnito commented Feb 4, 2019 •
edited
Loading

The output of yarn run audit when it finds new vulnerabilities is:

screenshot 2019-02-04 at 21 42 27

Note: bfj (big-friendly-JSON) dependency is part of CRA 2.x too.

@Gnito Gnito force-pushed the add-audit-script branch 5 times, most recently from b0082ae to 6c90719 Compare February 4, 2019 19:51
OtterleyW
OtterleyW approved these changes Feb 5, 2019
View reviewed changes
Copy link
Contributor

@OtterleyW OtterleyW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

kpuputti
kpuputti reviewed Feb 5, 2019
View reviewed changes
scripts/audit.js Outdated Show resolved Hide resolved
kpuputti
kpuputti reviewed Feb 5, 2019
View reviewed changes
.nsprc Outdated Show resolved Hide resolved
@Gnito Gnito merged commit 4a36f4a into master Feb 5, 2019
@Gnito Gnito deleted the add-audit-script branch February 5, 2019 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kpuputti kpuputti kpuputti left review comments

@OtterleyW OtterleyW OtterleyW approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Gnito @kpuputti @OtterleyW