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

feat: bump up cosmwasm from v1.0.0 to v1.1.9 #256

Merged
merged 434 commits into from
Jan 10, 2023

Conversation

zemyblue
Copy link
Member

@zemyblue zemyblue commented Dec 14, 2022

Description

major changes

Add

  • cosmwasm-std: Implement PartialEq for Binary and u8 arrays.
  • cosmwasm-std: Implement BankQuery::Supply to allow querying the total supply of a native token.
  • cosmwasm-std: Add QuerierWrapper::query_wasm_contract_info
  • cosmwasm-check: This is a new binary package that allows running various CosmWasm compatibility checks on compiled .wasm files.

Changes

  • Upgrade Wasmer from 2.2.1 to 2.3.0
  • all: Bump a few dependency versions to make the codebase compile with -Zminimal-versions
  • cosmwasm-std: Enable the abort feature by default. This provides more helpful panic messages via a custom panic handler.
  • cosmwasm-crypto: Bumped k256 0.10.4 -> 0.11 and digest 0.9 -> 0.10
  • cosmwasm-profiler: Package was removed
  • cosmwasm-schema: schema generation is now locked to produce strictly draft-07 schemas

Fixed

  • cosmwasm-schema: The QueryResponses derive macro now supports QueryMsgs with generics.

Types of changes

  • Bug fix (changes which fixes an issue)
  • New feature (changes which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • ETC (build, ci, docs, perf, refactor, style, test)

Checklist

uint and others added 30 commits August 15, 2022 13:38
Implement PartialEq for references on all math types - LHS and tests
`Uint256`/`Uint512` - implement `checked_div_euclid`
…eference-math-types

Implement `PartialEq` for references on all math types
Added deprecation message to check_contract.
Use clippy from Rust 1.63.0 in CI
* cosmwasm/v1.1.9: (340 commits)
  Set version: 1.1.9
  Update CHANGELOG formatting
  Add CHANGELOG entry for issue #1527
  fix: issue #1527
  Add test to reproduce missing QueryResponses path
  CHANGELOG update
  Set version: 1.1.8
  run .md formatter
  QueryResponses: no default type in gen'd impl block
  Set version: 1.1.6
  Upgrade clippy CI job to 1.65.0
  Adapt codebase to clippy 1.65.0
  Use resolver version 2 in workspace
  Add comment regarding rounding behaviour to multiply_ratio/checked_multiply_ratio
  Update schemas
  Document type of gas that is used for gas limits
  Update CHANGELOG.md
  CHANGELOG entry
  Make QueryResponses tests clearer
  schema: refactor and docs
  ...
@zemyblue zemyblue marked this pull request as draft December 14, 2022 11:33
@CLAassistant
Copy link

CLAassistant commented Dec 14, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 5 committers have signed the CLA.

✅ zemyblue
❌ larry0x
❌ webmaster128
❌ uint
❌ chris-ricketts
You have signed the CLA already but the status is still pending? Let us recheck it.

@zemyblue zemyblue self-assigned this Dec 14, 2022
…d update `Cargo.lock` of each contracts.

Signed-off-by: zemyblue <zemyblue@gmail.com>
Signed-off-by: zemyblue <zemyblue@gmail.com>
@zemyblue
Copy link
Member Author

@shiki-tak , @loloicci , is there any problem if I change the version to 1.1.9-0.7.0?

Comment on lines 45 to 65
{
"description": "This calls into the native bank module for querying the total supply of one denomination. It does the same as the SupplyOf call in Cosmos SDK's RPC API. Return value is of type SupplyResponse.",
"type": "object",
"required": [
"supply"
],
"properties": {
"supply": {
"type": "object",
"required": [
"denom"
],
"properties": {
"denom": {
"type": "string"
}
}
}
},
"additionalProperties": false
},
Copy link
Member Author

Choose a reason for hiding this comment

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

We need to check why this change should be removed for passing ci.

Copy link
Contributor

@loloicci loloicci left a comment

Choose a reason for hiding this comment

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

CHANGELOG is auto-updated when the version is updated in this repository. So Unreleased section is not needed.

Signed-off-by: zemyblue <zemyblue@gmail.com>
Copy link
Member

@da1suk8 da1suk8 left a comment

Choose a reason for hiding this comment

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

The .circleci/config.yml is empty, is there a reason why it has been added?

The rest is ok.

Copy link
Contributor

@shiki-tak shiki-tak left a comment

Choose a reason for hiding this comment

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

LGTM except for comments in the document

@@ -0,0 +1,580 @@
# IBC interfaces for CosmWasm contracts
Copy link
Contributor

Choose a reason for hiding this comment

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

IBC.md and MIGRATE.md appear to be unnecessary for our repository as they are linked from the README.

https://github.com/line/cosmwasm/blob/main/README.md

Copy link
Member Author

Choose a reason for hiding this comment

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

OK. I'll remove it. Thank you.

@zemyblue
Copy link
Member Author

The .circleci/config.yml is empty, is there a reason why it has been added?

The rest is ok.

Oh, I did remove it. But I think it comes alive again when merging. I'll remove again. Thank you.

Signed-off-by: zemyblue <zemyblue@gmail.com>
@zemyblue zemyblue merged commit 6ea2dfb into Finschia:main Jan 10, 2023
zemyblue added a commit to zemyblue/wasmvm that referenced this pull request Jan 10, 2023
…#256 PR)

Signed-off-by: zemyblue <zemyblue@gmail.com>
Expotential108 added a commit to Expotential108/Binding-cosmwasm that referenced this pull request Mar 12, 2023
…#256 PR)

Signed-off-by: zemyblue <zemyblue@gmail.com>
zemyblue added a commit to zemyblue/wasmvm that referenced this pull request Mar 13, 2023
…#256 PR)

Signed-off-by: zemyblue <zemyblue@gmail.com>
zemyblue added a commit to Finschia/wasmvm that referenced this pull request Mar 28, 2023
* chore: update latest `line/cosmwasm` commit (merged Finschia/cosmwasm#256 PR)

Signed-off-by: zemyblue <zemyblue@gmail.com>

* add library

* add changelog

* ci: add ci test about compiled internal libraries.

Signed-off-by: zemyblue <zemyblue@gmail.com>

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>
loloicci added a commit to Finschia/wasmvm that referenced this pull request Jun 14, 2023
* fix: wrong tag reference (v1.1.1-0.11.2) (#95)

* chore: update latest `line/cosmwasm` commit (merged Finschia/cosmwasm#256 PR)

Signed-off-by: zemyblue <zemyblue@gmail.com>

* add library

* add changelog

* ci: add ci test about compiled internal libraries.

Signed-off-by: zemyblue <zemyblue@gmail.com>

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>

* feat: add codeowners file (#100)

* fix: fix a test for rustc 1.68 or later (#108)

* build: replace line modules with Finschia's (#109)

* build: change depending cosmwasm and repository info to finschia

* build: replace module name line/wasmvm with Finschia/wasmvm

* docs: replace "line" with "Finschia" in docs and scripts related docs

* docs: change copylight

* build: rename docker containers from line/* to finschia/*

* docs: replace lbm-sdk with finschia-sdk

* docs: replace mail address and repository name

* docs: replace lbm-sdk with finschia-sdk in comments

* chore: update cargo author information. (#112)

* chore: update cargo author information.

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: update changelog

Signed-off-by: zemyblue <zemyblue@gmail.com>

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>
Co-authored-by: zemyblue <zemyblue@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.