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

imp(core-host-types): add new storage paths #1069

Merged
merged 13 commits into from
Feb 1, 2024

Conversation

rnbguy
Copy link
Collaborator

@rnbguy rnbguy commented Jan 30, 2024

Closes: #1065

Description

Adds the following paths in the ibc Path type.


PR author checklist:

  • Added changelog entry, using unclog.
  • Added tests.
  • Linked to GitHub issue.
  • Updated code comments and documentation (e.g., docs/).
  • Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

  • Reviewed Files changed in the GitHub PR explorer.
  • Manually tested (in case integration/unit/mock tests are absent).

Copy link

codecov bot commented Jan 30, 2024

Codecov Report

Attention: 43 lines in your changes are missing coverage. Please review.

Comparison is base (4a0990f) 67.07% compared to head (36202c0) 66.78%.
Report is 1 commits behind head on main.

Files Patch % Lines
ibc-core/ics24-host/types/src/path.rs 59.43% 43 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1069      +/-   ##
==========================================
- Coverage   67.07%   66.78%   -0.30%     
==========================================
  Files         204      204              
  Lines       20543    20473      -70     
==========================================
- Hits        13780    13672     -108     
- Misses       6763     6801      +38     

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

Copy link
Member

@Farhad-Shabani Farhad-Shabani left a comment

Choose a reason for hiding this comment

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

Just a nit, then all set!

ibc-core/ics24-host/types/src/path.rs Outdated Show resolved Hide resolved
@Farhad-Shabani Farhad-Shabani changed the title imp(host): add new storage paths imp(ibc-core-host-types): add new storage paths Feb 1, 2024
Copy link
Member

@Farhad-Shabani Farhad-Shabani left a comment

Choose a reason for hiding this comment

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

And a unclog please!

@Farhad-Shabani Farhad-Shabani changed the title imp(ibc-core-host-types): add new storage paths imp(core-host-types): add new storage paths Feb 1, 2024
@Farhad-Shabani Farhad-Shabani merged commit ba7b90c into main Feb 1, 2024
15 checks passed
@Farhad-Shabani Farhad-Shabani deleted the rano/host/add-storage-paths branch February 1, 2024 19:02
@rnbguy rnbguy restored the rano/host/add-storage-paths branch February 2, 2024 09:57
@rnbguy rnbguy deleted the rano/host/add-storage-paths branch February 2, 2024 09:57
Farhad-Shabani pushed a commit that referenced this pull request Sep 9, 2024
* add missing paths

* update Path enum with new paths

* update parse functions

* add next_sequence path parser

* add unit tests

* update names and path format

* update parsing functions

* update tests

* use rstest cases for Path::try_from

* add comment in rstest

* consistent with old tests

* update rstest test names

* add changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

imp(ics24-host): missing storage paths
2 participants