Skip to content
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

first version of v4 client integration in prism-go-client #133

Merged

Conversation

yannickstruyf3
Copy link
Contributor

Adds a new v4Client that holds v4 api instances.

tuxtof
tuxtof previously approved these changes Aug 31, 2023
Copy link
Contributor

@tuxtof tuxtof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

thunderboltsid
thunderboltsid previously approved these changes Aug 31, 2023
@thunderboltsid
Copy link
Contributor

Please add a Changelog entry too

@yannickstruyf3
Copy link
Contributor Author

Added Changelog entry

@tuxtof tuxtof force-pushed the feat/v4-client-initial branch from 4c6c2e1 to 6468b32 Compare September 9, 2023 02:29
@codecov-commenter
Copy link

Codecov Report

Merging #133 (6468b32) into main (c435e65) will increase coverage by 1.34%.
The diff coverage is 84.41%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main     #133      +/-   ##
==========================================
+ Coverage   52.96%   54.30%   +1.34%     
==========================================
  Files          19       20       +1     
  Lines        1722     1799      +77     
==========================================
+ Hits          912      977      +65     
- Misses        810      822      +12     
Files Changed Coverage Δ
v4/v4.go 84.41% <84.41%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tuxtof tuxtof merged commit 72182cd into nutanix-cloud-native:main Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants