crypto-org-chain / chain-main

Cronos POS Chainโ›“: Croeseid Testnet and beyond development
https://cronos-pos.org/
Other
496 stars 352 forks source link

Problem: sdk 47 is not integrated #1023

Closed yihuang closed 6 months ago

yihuang commented 8 months ago

๐Ÿ‘ฎ๐Ÿป๐Ÿ‘ฎ๐Ÿป๐Ÿ‘ฎ๐Ÿป !!!! REFERENCE THE PROBLEM YOUR ARE SOLVING IN THE PR TITLE AND DESCRIBE YOUR SOLUTION HERE !!!! DO NOT FORGET !!!! ๐Ÿ‘ฎ๐Ÿป๐Ÿ‘ฎ๐Ÿป๐Ÿ‘ฎ๐Ÿป

PR Checklist:

Thank you for your code, it's appreciated! :)

codecov[bot] commented 8 months ago

Codecov Report

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

Project coverage is 32.95%. Comparing base (911b8a5) to head (4a273d1).

Files Patch % Lines
app/upgrades.go 66.66% 17 Missing :warning:
x/nft/types/query.pb.go 64.28% 10 Missing :warning:
app/test_helpers.go 30.76% 9 Missing :warning:
app/app.go 88.73% 6 Missing and 2 partials :warning:
x/nft/client/testutil/test_helpers.go 0.00% 8 Missing :warning:
x/nft-transfer/types/query.pb.go 62.50% 6 Missing :warning:
x/nft/types/tx.pb.go 75.00% 5 Missing :warning:
x/chainmain/client/cli/testnet.go 20.00% 4 Missing :warning:
x/icaauth/types/params.pb.go 25.00% 3 Missing :warning:
x/icaauth/types/query.pb.go 62.50% 3 Missing :warning:
... and 9 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1023 +/- ## ========================================== + Coverage 32.91% 32.95% +0.03% ========================================== Files 130 128 -2 Lines 20161 20001 -160 ========================================== - Hits 6637 6592 -45 + Misses 12461 12353 -108 + Partials 1063 1056 -7 ``` | [Flag](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | Coverage ฮ” | | |---|---|---| | [integration_tests](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `21.04% <64.20%> (+0.44%)` | :arrow_up: | | [integration_tests_byzantine](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `9.89% <62.36%> (+0.35%)` | :arrow_up: | | [integration_tests_gov](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `10.26% <62.73%> (+0.35%)` | :arrow_up: | | [integration_tests_grpc](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `10.24% <62.73%> (+0.35%)` | :arrow_up: | | [integration_tests_ibc](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `25.11% <67.52%> (+0.40%)` | :arrow_up: | | [integration_tests_ledger](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `10.13% <62.36%> (+0.35%)` | :arrow_up: | | [integration_tests_slow](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `10.13% <62.36%> (+0.35%)` | :arrow_up: | | [integration_tests_solomachine](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `9.92% <62.36%> (+0.35%)` | :arrow_up: | | [integration_tests_upgrade](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `10.24% <62.73%> (+0.35%)` | :arrow_up: | | [unit_tests](https://app.codecov.io/gh/crypto-org-chain/chain-main/pull/1023/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain) | `10.40% <58.27%> (-0.19%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-org-chain#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.