-
-
Notifications
You must be signed in to change notification settings - Fork 596
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
chore: update dependencies #1339
Conversation
Something broken, need investigate and fix it |
Will take a look. 👍 |
Test is passing locally but not here 😕 If anyone has any ideas about this, please suggest. |
Problem with |
nyc upgrade was the issue, breaking change, reverted it. 👍 Will enable dependabot after this goes in. |
Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon. |
@anshumanv please fix conflicts |
Let's merge it and continue development |
I think we should check each package for unnecessary dependencies and remove them, it's a simple task |
On it. 👍 |
Also, enabling dependabot. |
What kind of change does this PR introduce?
updates deps accross all packages.
Did you add tests for your changes?
Yes
If relevant, did you update the documentation?
Not needed
Summary
Does this PR introduce a breaking change?
Shouldn't ideally
Other information
NA