-
-
Notifications
You must be signed in to change notification settings - Fork 152
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
feat:(e2e) added test case for mevn info command #141
feat:(e2e) added test case for mevn info command #141
Conversation
Thanks for opening this pull request! Please check out our contributing guidelines. |
@jamesgeorge007 you can review the code ! Thanks. |
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.
This is the e2e
approach.
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.
Revert the lock
file changes.
@jamesgeorge007 may be the changes are due to some npm issue? I can't revert it back.I can use some help,Thanks ! |
e771e33
to
36591b5
Compare
refactor: change let to const Co-authored-by: Thakur Karthik <[email protected]>
36591b5
to
f83663a
Compare
Description
This pr on successful merge adds a test case for
mevn info
cli commandAddresses #155
Type of change
Please delete options that are not relevant.
Checklist: