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

Update acorn to 7.1.1 #56

Closed
wants to merge 1 commit into from
Closed

Update acorn to 7.1.1 #56

wants to merge 1 commit into from

Conversation

EmmaGoodliffe
Copy link

Fixes following error from running npm audit:

  Moderate        Regular Expression Denial of Service                          

  Package         acorn                                                         

  Patched in      >=7.1.1                                                       

  Dependency of   acorn-globals                                        

  Path            acorn-globals > acorn                               

  More info       https://npmjs.com/advisories/1488  

@tunnckoCore
Copy link

Bot already opened a PR #50.

@EmmaGoodliffe
Copy link
Author

Awesome! Thank you!

@tunnckoCore
Copy link

tunnckoCore commented Mar 7, 2020

In anyway, please don't delete your branch since I suggested it as install method in several places 😆


Oh, you deleted it. Anyway :D

This should work too.

npm i ForbesLindesay/acorn-globals#greenkeeper/acorn-7.1.1

@EmmaGoodliffe
Copy link
Author

So sorry! I didn’t see this message. But thank you for your responsiveness! I really appreciate you taking the time to fix the problem

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