-
Notifications
You must be signed in to change notification settings - Fork 54
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
Updating all dependencies #47
base: master
Are you sure you want to change the base?
Conversation
@constverum PTAL |
Confirmed this worked great for my project. :) |
@mor10 This probably can't be merged because it switched from yarn to npm, would you be willing to merge a PR that bumps everything without doing the switch? |
could you tell us what's reason to save yarn? |
Not really but there is no reason to change this as it was the maintainer's preferred choice and is not in scope for a pull request that just updates dependency versions. |
Why am I included in this? It's not my PR and not my repo. |
Oh sorry, I thought you're a contributor judging from the other conversations. Guess we'll have to wait for @constverum then! |
I'm just a regular person with opinions. I have no power (in this project). Hope the PR gets merged though. |
I updated all dependencies for fix npm audit vulnerabilities.