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

Upgrade dependencies #85

Merged
merged 4 commits into from
Nov 1, 2019
Merged

Upgrade dependencies #85

merged 4 commits into from
Nov 1, 2019

Conversation

stof
Copy link
Contributor

@stof stof commented Oct 29, 2019

This upgrades dependencies to the latest version (except for marked, which is handled in #84 with some discussion).

This will remove github vulnerability alerts on this repo (the lock file does not reference old releases of set-value, cryptiles and randomatic anymore), and makes yarn audit on this repo go from 8892 vulnerabilities (cryptiles is reported many times for instance, due to it being required in many places in the dependency tree) to only 1 (the marked one of #84)

@stof stof requested a review from ndelangen October 29, 2019 11:45
"eslint-plugin-jsx-a11y": "^6.2.1",
"eslint-plugin-prettier": "^3.1.0",
"eslint-plugin-react": "^7.13.0",
"eslint-plugin-react-hooks": "^1.7.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this one is a new peer dependency of eslint-config-airbnb

@torquemad
Copy link

@stof I would also like to see this resolved, looks like your PR has been approved if you want to merge it in :)

@ndelangen ndelangen merged commit f6b3c6a into master Nov 1, 2019
@ndelangen ndelangen deleted the upgrade_deps branch November 1, 2019 06:45
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.

3 participants