Skip to content

New API for MeshInfo #1029

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

Merged
merged 14 commits into from
Jul 13, 2023
Merged

New API for MeshInfo #1029

merged 14 commits into from
Jul 13, 2023

Conversation

rafacanton
Copy link
Contributor

@rafacanton rafacanton commented Jul 13, 2023

  • Simplification of API for the MeshInfo (no need to pass the type of a property to retrieve it)
  • Elimination of the body_name, zone_name and topology scenarios of the MeshInfo tests to avoid failing pipelines while doing the next server migration (will be reverted)

@rafacanton rafacanton self-assigned this Jul 13, 2023
@rafacanton rafacanton added the enhancement New feature or request label Jul 13, 2023
@rafacanton rafacanton changed the title Rcanton/mesh info new api New API for MeshInfo Jul 13, 2023
@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Merging #1029 (372b37f) into master (e37485e) will decrease coverage by 1.23%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1029      +/-   ##
==========================================
- Coverage   88.54%   87.32%   -1.23%     
==========================================
  Files          77       77              
  Lines        8727     8731       +4     
==========================================
- Hits         7727     7624     -103     
- Misses       1000     1107     +107     

@rafacanton rafacanton added this to the v0.9.0 milestone Jul 13, 2023
@rafacanton rafacanton enabled auto-merge (squash) July 13, 2023 08:30
@rafacanton rafacanton merged commit 740ca61 into master Jul 13, 2023
@rafacanton rafacanton deleted the rcanton/mesh_info_new_api branch July 13, 2023 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants