Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Refactor: break down DeploySmartContract into smaller methods to improve readability#3448

Open
gldeng wants to merge 1 commit intofeature/deterministic-address from
refactor/DeploySmartContract-core-function
Open

Refactor: break down DeploySmartContract into smaller methods to improve readability #3448
gldeng wants to merge 1 commit intofeature/deterministic-address from
refactor/DeploySmartContract-core-function

Conversation

@gldeng
Copy link
Contributor

@gldeng gldeng commented Sep 21, 2023

No description provided.

Copy link

codecov bot commented Sep 21, 2023
edited
Loading

Codecov Report

Merging #3448 (7c6d5d6) into feature/deterministic-address (a705e46) will decrease coverage by 9.48%.
The diff coverage is 74.38%.

Additional details and impacted files

Impacted file tree graph

@@ Coverage Diff @@
## feature/deterministic-address #3448 +/- ##
=================================================================
- Coverage 90.72% 81.25% -9.48% 
=================================================================
 Files 663 664 +1 
 Lines 25388 25444 +56 
 Branches 2280 2282 +2 
=================================================================
- Hits 23033 20674 -2359 
- Misses 2239 4607 +2368 
- Partials 116 163 +47 
Files Changed Coverage Δ
...ntract/AElf.Contracts.Genesis/BasicContractZero.cs 22.25% <47.61%> (-61.73%) ⬇️
contract/AElf.Contracts.Genesis/Extensions.cs 77.02% <77.02%> (ø)
...AElf.Contracts.Genesis/BasicContractZero_Helper.cs 40.00% <88.46%> (-57.03%) ⬇️

... and 125 files with indirect coverage changes

Copy link

vizipi bot commented Sep 21, 2023

Pull request analysis by VIZIPI

Below you will find who is the most qualified team member to review your code.
This analysis includes his/her work on the code included in this Pull request, in addition to their experience in code affected by these changes ( partly found within the list of potential missing files below ) Feedback always welcome

Reviewers with knowledge related to these changes

Match % Person Commit Count Common Files
66.67 % jason-hoopox 29 2
66.67 % ZhangXin 20 2
66.67 % kaiwenzhengaelf 17 2
66.67 % ssun0121 14 2
66.67 % YueZh127 2 2
66.67 % AbigailDeng 1 2

Potential missing files from this Pull request

files commonly committed with a subset of this pr, but not committed this time. (click to collapse)
FilePercentilerate
...test/AElf.Contracts.Genesis.Tests/GenesisContractAuthTest 50.00 %52 out of 104 times
protobuf/acs0.proto 49.04 %51 out of 104 times
contract/AElf.Contracts.Genesis/BasicContractZeroState.cs 42.31 %44 out of 104 times

Committed file ranks

(click to expand)
  • 98.54%[contract/AElf.Contracts.Genesis/BasicContractZero_Helper.cs]
  • 99.82%[contract/AElf.Contracts.Genesis/BasicContractZero.cs]
  • 0.00%[contract/AElf.Contracts.Genesis/Extensions.cs]
  • Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Reviewers

    @jason-aelf jason-aelf Awaiting requested review from jason-aelf

    @zhxymh zhxymh Awaiting requested review from zhxymh

    @kaiwenzhengaelf kaiwenzhengaelf Awaiting requested review from kaiwenzhengaelf

    @ssun0121 ssun0121 Awaiting requested review from ssun0121

    @YueZh127 YueZh127 Awaiting requested review from YueZh127

    @AbigailDeng AbigailDeng Awaiting requested review from AbigailDeng

    Assignees

    No one assigned

    Labels

    None yet

    Projects

    None yet

    Milestone

    No milestone

    Development

    Successfully merging this pull request may close these issues.

    1 participant

    Comments

    AltStyle によって変換されたページ (->オリジナル) /