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

wasm40 upgrade #3234

Merged
merged 10 commits into from
Aug 28, 2023
Merged

wasm40 upgrade #3234

merged 10 commits into from
Aug 28, 2023

Conversation

ylsGit
Copy link
Contributor

@ylsGit ylsGit commented Jul 29, 2023

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

  • Wrote tests

  • Updated relevant documentation (docs/)

  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md

  • Reviewed Files changed in the github PR explorer


For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge PR #XYZ: [title]" (coding standards)

chengzhinei and others added 5 commits July 28, 2023 16:41
* add gas table

* change depend wasmvm
* adapt to  wasmvm v1.2.4

* go mod tidy

* support create contract by contract

* support  create2

* support create contract by codeID

* change contract address2

* update wasmvm version

* change GoAPI as local var: fix create smb

* add CallCreateDepth

* fix merge error

---------

Co-authored-by: ylsGit <[email protected]>
Co-authored-by: ylsGit <[email protected]>
* adapt to  wasmvm v1.2.4

* go mod tidy

* switch to wasmvm v1.2.4

* replace the dependce

* add file

* add ut

* add transfer

* modity the go.mod for recursion

* add getCallerInfoFunc transferCoinsFunc in api

* merge

---------

Co-authored-by: ylsGit <[email protected]>
Co-authored-by: ylsGit <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2023

Codecov Report

Merging #3234 (9d15b62) into dev (2bee5f9) will decrease coverage by 0.03%.
The diff coverage is 31.95%.

❗ Current head 9d15b62 differs from pull request most recent head a6e78be. Consider uploading reports for the commit a6e78be to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##              dev    #3234      +/-   ##
==========================================
- Coverage   36.56%   36.54%   -0.03%     
==========================================
  Files         445      446       +1     
  Lines       46705    46857     +152     
==========================================
+ Hits        17079    17123      +44     
- Misses      28360    28463     +103     
- Partials     1266     1271       +5     
Files Changed Coverage Δ
x/wasm/module.go 15.87% <0.00%> (-0.26%) ⬇️
x/wasm/types/errors.go 13.33% <ø> (ø)
x/wasm/types/store_adapter.go 0.00% <0.00%> (ø)
x/wasm/types/types.go 38.46% <0.00%> (-1.34%) ⬇️
x/wasm/types/validation.go 63.15% <0.00%> (-36.85%) ⬇️
x/wasm/keeper/api.go 19.35% <4.16%> (-52.08%) ⬇️
app/app.go 80.76% <20.00%> (-0.56%) ⬇️
x/wasm/keeper/cross_contract_call.go 34.04% <34.04%> (ø)
x/wasm/keeper/keeper.go 76.18% <45.20%> (-2.88%) ⬇️
x/wasm/keeper/contract_keeper.go 56.09% <100.00%> (+1.09%) ⬆️

ylsGit and others added 3 commits August 2, 2023 16:21
* add venus7 for wasm upgrade

* upgrade go mod

* upgrade go mod

* upgrade go mod wasmvm version
@lyh169 lyh169 changed the title Wasmvm 130 wasm40 upgrade Aug 28, 2023
@ylsGit ylsGit marked this pull request as ready for review August 28, 2023 07:04
@KamiD KamiD merged commit 3301eae into dev Aug 28, 2023
7 checks passed
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.

6 participants