Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

derive PartialEq, Eq for MsgEnvelope #847

Merged
merged 2 commits into from
Sep 1, 2023
Merged

derive PartialEq, Eq for MsgEnvelope #847

merged 2 commits into from
Sep 1, 2023

Conversation

plafer
Copy link
Contributor

@plafer plafer commented Sep 1, 2023

No description provided.

Verified

This commit was signed with the committer’s verified signature.
renovate-bot Mend Renovate
@plafer plafer marked this pull request as ready for review September 1, 2023 13:19
@codecov
Copy link

codecov bot commented Sep 1, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (2c28380) 71.59% compared to head (24b227c) 71.59%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #847   +/-   ##
=======================================
  Coverage   71.59%   71.59%           
=======================================
  Files         124      124           
  Lines       15142    15142           
=======================================
  Hits        10841    10841           
  Misses       4301     4301           
Files Changed Coverage Δ
crates/ibc/src/core/ics02_client/msgs.rs 94.11% <100.00%> (ø)
...s/ibc/src/core/ics02_client/msgs/upgrade_client.rs 75.34% <100.00%> (ø)
crates/ibc/src/core/msgs.rs 1.04% <100.00%> (ø)

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

@plafer plafer merged commit 2d610b6 into main Sep 1, 2023
@plafer plafer deleted the plafer/msgenvelope-eq branch September 1, 2023 13:30
@Farhad-Shabani Farhad-Shabani added this to the v0.45.0 milestone Sep 7, 2023
Farhad-Shabani pushed a commit that referenced this pull request Sep 9, 2024
* derive PartialEq, Eq

* changelog
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants