-
Notifications
You must be signed in to change notification settings - Fork 28
Conversation
Signed-off-by: Andrew W. Harn <andrew.harn@broadcom.com>
Codecov ReportBase: 82.70% // Head: 82.70% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## zowe-v1-lts zowe/imperative#879 +/- ##
===============================================
- Coverage 82.70% 82.70% -0.01%
===============================================
Files 163 163
Lines 7949 7948 -1
Branches 1455 1454 -1
===============================================
- Hits 6574 6573 -1
Misses 1371 1371
Partials 4 4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: Andrew W. Harn <andrew.harn@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.
The "code smells" are just because of isNullOrUndefined
statements which were there before, so nothing new. LGTM 👍
SonarCloud Quality Gate failed. |
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.
to make sonarcloud happy again, should we replace the isNullOrUndefined
in this file:
packages/cmd/src/profiles/CliProfileManager.ts
I'm happy to merge as-is since the changes LGTM though 😋
merging as-is since we already have this issue: |
Release succeeded for the The following packages have been published:
Powered by Octorelease 🚀 |
Backport the code smell fixes to V1 LTS