Skip to content
This repository has been archived by the owner on Nov 13, 2023. It is now read-only.

[Backport] Fix code smells #879

Merged
merged 3 commits into from
Aug 29, 2022
Merged

[Backport] Fix code smells #879

merged 3 commits into from
Aug 29, 2022

Conversation

awharn
Copy link
Member

@awharn awharn commented Aug 26, 2022

Backport the code smell fixes to V1 LTS

Signed-off-by: Andrew W. Harn <andrew.harn@broadcom.com>
@codecov
Copy link

codecov bot commented Aug 26, 2022

Codecov Report

Base: 82.70% // Head: 82.70% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (7eff9cd) compared to base (2d15654).
Patch coverage: 100.00% of modified lines in pull request are covered.

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              
Impacted Files Coverage Δ
packages/cmd/src/profiles/CliProfileManager.ts 96.42% <100.00%> (-0.02%) ⬇️
packages/cmd/src/response/CommandResponse.ts 97.04% <100.00%> (ø)

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.
📢 Do you have feedback about the report comment? Let us know in this issue.

Signed-off-by: Andrew W. Harn <andrew.harn@broadcom.com>
@awharn awharn marked this pull request as ready for review August 26, 2022 18:58
Copy link
Member

@t1m0thyj t1m0thyj left a 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 👍

@gejohnston gejohnston self-requested a review August 26, 2022 22:27
@sonarcloud
Copy link

sonarcloud bot commented Aug 29, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell E 10 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

Copy link
Member

@zFernand0 zFernand0 left a 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 😋

@zFernand0
Copy link
Member

merging as-is since we already have this issue:

@zFernand0 zFernand0 merged commit 5cf170b into zowe-v1-lts Aug 29, 2022
@zFernand0 zFernand0 deleted the fix-code-smells-v1 branch August 29, 2022 17:32
@zFernand0 zFernand0 added the release-patch Indicates a patch to existing code has been applied label Aug 29, 2022
@github-actions
Copy link

Release succeeded for the zowe-v1-lts branch. 🎉

The following packages have been published:

  • npm: @zowe/imperative@4.18.9

Powered by Octorelease 🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
no-changelog release-patch Indicates a patch to existing code has been applied released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants