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

Refactor evm address #1200

Merged
merged 10 commits into from
May 17, 2022
Merged

Refactor evm address #1200

merged 10 commits into from
May 17, 2022

Conversation

bysomeone
Copy link
Collaborator

  • evm地址模块重构, 适配chain33 多地址格式

  • 支持evm地址类型单独配置, 区别框架默认地址类型

@bysomeone bysomeone force-pushed the refactor-evm-address branch 2 times, most recently from 61edd6c to e152829 Compare May 11, 2022 09:26
@codecov
Copy link

codecov bot commented May 11, 2022

Codecov Report

Merging #1200 (c1e08f7) into master (7c34a38) will increase coverage by 0.12%.
The diff coverage is 67.92%.

❗ Current head c1e08f7 differs from pull request most recent head 631ff03. Consider uploading reports for the commit 631ff03 to get more accurate results

@@            Coverage Diff             @@
##           master    #1200      +/-   ##
==========================================
+ Coverage   56.02%   56.15%   +0.12%     
==========================================
  Files         211      211              
  Lines       35100    35116      +16     
==========================================
+ Hits        19666    19719      +53     
+ Misses      13690    13653      -37     
  Partials     1744     1744              
Impacted Files Coverage Δ
plugin/dapp/evm/executor/vm/runtime/contracts.go 79.40% <60.00%> (-0.30%) ⬇️
plugin/dapp/evm/executor/vm/common/address.go 69.44% <68.75%> (+41.31%) ⬆️
plugin/dapp/evm/executor/vm/common/bytes.go 0.00% <0.00%> (-7.41%) ⬇️
plugin/consensus/qbft/consensus_state.go 60.32% <0.00%> (+0.18%) ⬆️
plugin/dapp/evm/executor/vm/common/hash.go 31.25% <0.00%> (+31.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7c34a38...631ff03. Read the comment docs.

@bysomeone bysomeone force-pushed the refactor-evm-address branch 2 times, most recently from 6093596 to 33d0058 Compare May 12, 2022 01:46
@bysomeone bysomeone changed the title WIP Refactor evm address Refactor evm address May 12, 2022
@bysomeone bysomeone changed the title Refactor evm address WIP Refactor evm address May 12, 2022
@bysomeone bysomeone changed the title WIP Refactor evm address Refactor evm address May 16, 2022
@vipwzw vipwzw merged commit 565e980 into 33cn:master May 17, 2022
@33cn
Copy link
Owner

33cn commented May 27, 2022

🎉 This PR is included in version 1.67.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@33cn 33cn added the released label May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants