-
Notifications
You must be signed in to change notification settings - Fork 99
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
fix(v3): Fix sessions disappearing from trees caused by zowe.extRefresh
#3009
Conversation
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #3009 +/- ##
==========================================
+ Coverage 92.74% 92.82% +0.08%
==========================================
Files 107 107
Lines 10882 10887 +5
Branches 2360 2358 -2
==========================================
+ Hits 10092 10106 +14
+ Misses 788 779 -9
Partials 2 2 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
185601e
to
b8c96d0
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.
One minor comment, but otherwise this looks dandy.
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
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.
I was able to confirm that there was an issue with my Zowe team config's secure array. After resetting my config file and preparing my environment, I was unable to reproduce the previous scenario I encountered.
LGTM, thanks Timothy for the fix!
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
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! 😋
I'm guessing it's ok for this port-PR to be merged before the V2 one since we don't need to verify Theia support for V3? 😋
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.
Nice!
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
|
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.
I started reviewing this one first and ended up approving the v2 implementation by accident - oops 😅 Both PRs look good though.
LGTM, thanks Timothy!
Proposed changes
Fixes #2906
Release Notes
Milestone: v3
Changelog: Fixed issue where creating new team configuration file could cause Zowe Explorer to crash, resulting in all sessions disappearing from trees
Types of changes
Checklist
General
yarn workspace vscode-extension-for-zowe vscode:prepublish
pnpm --filter vscode-extension-for-zowe vscode:prepublish
Code coverage
Deployment
Further comments