-
Notifications
You must be signed in to change notification settings - Fork 493
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
test(stack/storage): updates dependencies and add missing tests #2298
Conversation
DeepCode's analysis on #9fd345 found:
💬 This comment has been generated by the DeepCode bot, installed by the owner of the repository. The DeepCode bot protects your repository by detecting and commenting on security vulnerabilities or other critical issues. |
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.
Looks good, you just put the same dep twice
packages/stack/storage/package.json
Outdated
"embark-solo": "^5.2.3", | ||
"embark-utils": "^5.3.0-nightly.0", |
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.
You put embark-utils both in dependencies and dev-deps, you can remove the latter then.
6d796f4
to
7215d0e
Compare
b84ee38
to
c7c533e
Compare
Thanks @jrainville and @michaelsbradleyjr for you comments. This is now updated accordingly and rebased on top of latest master |
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.
LGTM 👍
c7c533e
to
9d620bd
Compare
Needs a rebase. |
9d620bd
to
4eed997
Compare
4eed997
to
9fd3456
Compare
No description provided.