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

Fix abstract rest client (2 of 4) (Imperative V1) #1016

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

awharn
Copy link
Member

@awharn awharn commented Oct 4, 2023

What It Does
Fixes the abstract rest client's implementation of normalization

How to Test

Review Checklist
I certify that I have:

Additional Comments

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

codecov bot commented Oct 4, 2023

Codecov Report

All modified lines are covered by tests ✅

Files Coverage Δ
packages/rest/src/client/AbstractRestClient.ts 86.54% <100.00%> (+2.77%) ⬆️

... and 1 file with indirect coverage changes

📢 Thoughts on this report? Let us know!.

@sonarcloud
Copy link

sonarcloud bot commented Oct 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

88.2% 88.2% Coverage
0.0% 0.0% Duplication

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.

LGTM, thanks @awharn!

@awharn awharn merged commit ce42a78 into zowe-v1-lts Oct 5, 2023
16 checks passed
@awharn awharn deleted the fix-abstract-rest-client-v1 branch October 5, 2023 13:23
@awharn awharn restored the fix-abstract-rest-client-v1 branch October 5, 2023 13:23
@awharn awharn deleted the fix-abstract-rest-client-v1 branch October 5, 2023 13:23
@awharn awharn added the release-patch Indicates a patch to existing code has been applied label Oct 5, 2023
@github-actions
Copy link

github-actions bot commented Oct 5, 2023

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

The following packages have been published:

  • npm: @zowe/imperative@4.18.19

Powered by Octorelease 🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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.

4 participants