desmos-labs / desmos

Improving the well-being of users on social networks through the blockchain technology.
https://desmos.network
Apache License 2.0
104 stars 46 forks source link

chore: prepare v6.0.0 #1212

Closed RiccardoM closed 1 year ago

RiccardoM commented 1 year ago

Description

This PR prepares the codebase for the upcoming v6.0.0 release. The changes included are:

  1. Bumped the go module from github.com/desmos-labs/desmos/v5 to github.com/desmos-labs/desmos/v6.
  2. Added missing changeset entries, and fixed the messages on existing ones.
  3. Updated the on-chain upgrade workflow to test the upgrade from v5.2.0 to v6.0.0 using the chain state exported from the testnet

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.

I have...

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (b717680) 80.49% compared to head (491ee7d) 80.49%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1212 +/- ## ======================================= Coverage 80.49% 80.49% ======================================= Files 218 218 Lines 18287 18287 ======================================= Hits 14720 14720 Misses 2955 2955 Partials 612 612 ``` | [Files Changed](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs) | Coverage Δ | | |---|---|---| | [app/ante.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-YXBwL2FudGUuZ28=) | `53.84% <ø> (ø)` | | | [app/app.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-YXBwL2FwcC5nbw==) | `86.71% <ø> (ø)` | | | [app/app\_config.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-YXBwL2FwcF9jb25maWcuZ28=) | `50.00% <ø> (ø)` | | | [app/desmos/cmd/chainlink/create\_json.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-YXBwL2Rlc21vcy9jbWQvY2hhaW5saW5rL2NyZWF0ZV9qc29uLmdv) | `54.71% <ø> (ø)` | | | [app/encoding.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-YXBwL2VuY29kaW5nLmdv) | `0.00% <ø> (ø)` | | | [app/wasm\_config.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-YXBwL3dhc21fY29uZmlnLmdv) | `97.91% <ø> (ø)` | | | [tests/wasm/helpers.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-dGVzdHMvd2FzbS9oZWxwZXJzLmdv) | `91.89% <ø> (ø)` | | | [x/posts/abci.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-eC9wb3N0cy9hYmNpLmdv) | `100.00% <ø> (ø)` | | | [x/posts/keeper/alias\_functions.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-eC9wb3N0cy9rZWVwZXIvYWxpYXNfZnVuY3Rpb25zLmdv) | `69.71% <ø> (ø)` | | | [x/posts/keeper/attachments.go](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs#diff-eC9wb3N0cy9rZWVwZXIvYXR0YWNobWVudHMuZ28=) | `100.00% <ø> (ø)` | | | ... and [68 more](https://app.codecov.io/gh/desmos-labs/desmos/pull/1212?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=desmos-labs) | |

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

dadamu commented 1 year ago

Test on chain upgrade CI is failed because of timetout. Tested manually and it works properly. I think we should increase the timeout time to 1 hour or longer.

RiccardoM commented 1 year ago

@dadamu I have no idea why the workflow is failing. I have incremented this to 1 hour but it still has problems. I would guess something related to GitHub for some reason. Anyway, I would like to merge this if you approve it

dadamu commented 1 year ago

@RiccardoM Overall looks great, I think it is ready to go after this missing upgrade handler is added: https://github.com/desmos-labs/desmos/pull/1212#discussion_r1299682669

RiccardoM commented 1 year ago

@dadamu Thank, I missed that comment. I have now merged your code using #1216 :pray: