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

Feat: list relationships cmd #10898

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Aijeyomah
Copy link
Contributor

@Aijeyomah Aijeyomah commented May 8, 2024

Notes for Reviewers

This PR fixes #9844

  • List relationships experimental conmand

Signed commits

  • Yes, I signed my commits.

@github-actions github-actions bot added the component/mesheryctl CLI for Meshery label May 8, 2024
@Aijeyomah Aijeyomah changed the title Feat: list relationshipscmd Feat: list relationships cmd May 8, 2024
Copy link

github-actions bot commented May 8, 2024

@Aijeyomah Aijeyomah force-pushed the aijeyomah/feature/list-exp-relationship branch from fdac5d2 to 12c5c80 Compare June 3, 2024 00:37
@Aijeyomah Aijeyomah marked this pull request as ready for review June 3, 2024 00:41
@Aijeyomah Aijeyomah force-pushed the aijeyomah/feature/list-exp-relationship branch from 12c5c80 to 16cc475 Compare June 3, 2024 00:43
Copy link

codecov bot commented Jun 3, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 9.14%. Comparing base (63478b2) to head (16cc475).
Report is 118 commits behind head on master.

Files Patch % Lines
mesheryctl/pkg/utils/error.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #10898      +/-   ##
==========================================
- Coverage    9.15%    9.14%   -0.01%     
==========================================
  Files         146      146              
  Lines       19243    19249       +6     
==========================================
  Hits         1761     1761              
- Misses      17180    17186       +6     
  Partials      302      302              
Flag Coverage Δ
unittests 9.14% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

}

relationshipsResponse := &models.MeshmodelRelationshipsAPIResponse{}
err = json.Unmarshal(data, relationshipsResponse)
Copy link
Contributor

Choose a reason for hiding this comment

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

@Aijeyomah same for me, the error I posted in slack thread is on this line

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/mesheryctl CLI for Meshery
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[mesheryctl] New command exp relationship
3 participants