-
Notifications
You must be signed in to change notification settings - Fork 17
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: add stm annotations/增加STM注解 #103
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #103 +/- ##
===========================================
+ Coverage 13.18% 28.11% +14.92%
===========================================
Files 46 48 +2
Lines 3936 4091 +155
===========================================
+ Hits 519 1150 +631
+ Misses 3328 2741 -587
- Partials 89 200 +111
Flags with carried forward coverage won't be shown. Click here to find out more.
|
1c46467
to
6e55a6d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LTGM
关联的Issues (Related Issues)
close filecoin-project/venus#5316
改动 (Proposed Changes)
主要目的:
增加venus-wallet STM annotation.
实际修改:
💁♂️ :有部分重构是在编写测试用例的时候才发现需要重构的, 实在不太好单独拆分成不同的commit, 所以, 在commit上混到一起了.
附注 (Additional Info)
自查清单 (Checklist)
在你认为本 PR 满足被审阅的标准之前,需要确保 / Before you mark the PR ready for review, please make sure that: