-
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
Update dependencies #298
Update dependencies #298
Conversation
c2a988f
to
7cb8136
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Comandeer
There are a few packages that we can update, especially the typescript
, let's update it and we'll be ready to merge.
@KarolDawidziuk, thx for the review! I've bumped the deps, except the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @Comandeer for the improvements.
LGTM! 🎉
There is one failing test that we should check. It's can be related to an outdated version of the nightwatch
package, so we should check if fixing #287 will help.
51a946b
to
76f909b
Compare
No description provided.