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

Updated async to patched version #1672

Merged
merged 1 commit into from
Aug 15, 2019
Merged

Updated async to patched version #1672

merged 1 commit into from
Aug 15, 2019

Conversation

ajcrowe
Copy link
Contributor

@ajcrowe ajcrowe commented Jul 16, 2019

This patches async to a patched version containing the updated lodash package.

@botic
Copy link

botic commented Jul 29, 2019

Any changes to get this merged soon? Automated checks go crazy because of lodash's vulnerability / npm audit 🙃

DABH
DABH approved these changes Aug 15, 2019
View reviewed changes
Copy link
Contributor

@DABH DABH left a comment

Choose a reason for hiding this comment

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

seems fine, I can get this to master at least but not onto npm

@DABH DABH merged commit 9399f08 into winstonjs:master Aug 15, 2019
Mizumaki pushed a commit to Mizumaki/winston that referenced this pull request Jun 11, 2020
@DABH
Updated async to lodash patched version (winstonjs#1672)
33cd4cf
@Omrisnyk Omrisnyk mentioned this pull request Sep 6, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DABH DABH DABH 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
@ajcrowe @botic @DABH