Closed millken closed 4 months ago
Attention: Patch coverage is 63.52941%
with 62 lines
in your changes are missing coverage. Please review.
Project coverage is 76.94%. Comparing base (
cc526cd
) to head (e8d702a
). Report is 17 commits behind head on master.:exclamation: Current head e8d702a differs from pull request most recent head 7092b43
Please upload reports for the commit 7092b43 to get more accurate results.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Issues
2 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
1.3% Duplication on New Code
Description
In the last PR #4239, we have implemented
GetIdentifier()
.This which replaced all Bucket (CreateStake/Unstake/DepositToStake/Restake/ChangeCandidate) and IIP13 contractIndex places with the new
GetIdentifier()
Related PR: https://github.com/iotexproject/iotex-core/pull/4236 https://github.com/iotexproject/iotex-core/pull/4239 https://github.com/iotexproject/iotex-core/pull/4245 https://github.com/iotexproject/iotex-core/pull/4250
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: