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

Update packages with vulnerabilities #59

Merged
merged 1 commit into from
May 10, 2021
Merged

Update packages with vulnerabilities #59

merged 1 commit into from
May 10, 2021

Conversation

dhruvkb
Copy link
Member

@dhruvkb dhruvkb commented May 7, 2021

This PR updates packages via npm audit fix.

@dhruvkb dhruvkb requested review from zackkrida and obulat May 7, 2021 08:36
obulat
obulat approved these changes May 7, 2021
View reviewed changes
Copy link
Contributor

@obulat obulat left a comment

Choose a reason for hiding this comment

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

Great to get rid of so many vulnerabilities!

zackkrida
zackkrida approved these changes May 10, 2021
View reviewed changes
Copy link
Member

@zackkrida zackkrida left a comment

Choose a reason for hiding this comment

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

Great, if you look at the diff this only changes ~4 packages, so this was a case of some critical nested dependencies (mainly lodash) having vulnerabilities. Thanks for the fix.

@zackkrida zackkrida merged commit 2031bc4 into develop May 10, 2021
@zackkrida zackkrida deleted the vulnerabilities branch May 10, 2021 18:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@zackkrida zackkrida zackkrida approved these changes

@obulat obulat obulat 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
@dhruvkb @zackkrida @obulat