-
Notifications
You must be signed in to change notification settings - Fork 86
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: added method to validate the ssh connection #2340
Conversation
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2340 +/- ##
========================================
Coverage 91.13% 91.13%
========================================
Files 636 636
Lines 18002 18011 +9
Branches 3864 3759 -105
========================================
+ Hits 16406 16415 +9
Misses 1595 1595
Partials 1 1 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
Signed-off-by: Fernando Rijo Cedeno <[email protected]>
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, thanks Fernando!
Quality Gate passedIssues Measures |
This should be on public NPM after this workflow is done: |
What It Does
$
from the output of ssh commands at the sdk levelHow to Test
zowe uss issue ssh "ls /"
should display all folders/files with proper alignmentReview Checklist
I certify that I have:
Additional Comments
Related to: