-
Notifications
You must be signed in to change notification settings - Fork 656
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
Open shell
button starts the VM if needed
#3830
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3830 +/- ##
==========================================
- Coverage 88.95% 88.94% -0.01%
==========================================
Files 256 256
Lines 14584 14584
==========================================
- Hits 12973 12972 -1
- Misses 1611 1612 +1 ☔ View full report in Codecov by Sentry. |
ca75b7a
to
a2ecad8
Compare
728546d
to
37831d4
Compare
a2ecad8
to
8fab6a6
Compare
37831d4
to
c119f93
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.
LGTM! Tests pass and functions as expected
8fab6a6
to
0e909a0
Compare
c119f93
to
bb77155
Compare
2c63e3d
to
acf4d8f
Compare
bb77155
to
a9bd618
Compare
acf4d8f
to
7f7d7fd
Compare
a9bd618
to
859eef1
Compare
The base branch was changed.
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!
Now, if the VM is in a state from which it can be started (
Stopped
orSuspended
), theOpen shell
button will be enabled and it will start the VM before shelling into it.MULTI-1681