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

Dependabot/npm and yarn/acorn 7.1.1 #20

Merged
merged 1 commit into from Mar 20, 2020

Conversation

wu-lee
Copy link
Contributor

@wu-lee wu-lee commented Mar 16, 2020

Pending work addressing an upstream vulnerability in a dependency of ESLint.

GHSA-7fhm-mqm4-2wp7

ESlint has not yet fixed this, but work seems in progress:

eslint/eslint#13050

This will need to be revisited when it is fixed. In the mean time, we don't need to worry too much. ESLint is just a static analysis tool for finding bugs or bad EcmaScript practices in sea-map, and not part of the actual functionality of sea-map. Therefore this is not really a problem for users of sea-map, only a potential issue for developers who run ESLint on the code.

This fixes the problems with ESLint, as described here:

eslint/eslint#13050 (comment)

The problematic dependency `mkdirp` published a new version that bumps
`minimist` to a safe version.
@wu-lee
Copy link
Contributor Author

wu-lee commented Mar 20, 2020

See here, there is a workaround for this:

eslint/eslint#13050 (comment)

I shall rebase this branch to incorporate that instead.

@wu-lee wu-lee force-pushed the dependabot/npm_and_yarn/acorn-7.1.1 branch from 53552df to 460c8b5 Compare March 20, 2020 12:56
@wu-lee wu-lee merged commit eeea384 into master Mar 20, 2020
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.

None yet

1 participant