-
Notifications
You must be signed in to change notification settings - Fork 459
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: #4910 venus : add market api #4965
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4965 +/- ##
======================================
- Coverage 25% 25% -1%
======================================
Files 597 597
Lines 64991 65031 +40
======================================
- Hits 16575 16563 -12
- Misses 46254 46303 +49
- Partials 2162 2165 +3 |
30d388f
to
95aa71f
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.
第一个commit messager格式不太符合规范要求,其他挺好
aim to list piece storage
AddFsPieceStorage RemovePieceStorage GetPieceStorages
95aa71f
to
0f80d85
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
Related Issues
Proposed Changes
add market share api:
AddFsPieceStorage
ListPieceStorage
RemovePieceStorage
Additional Info
Checklist
Before you mark the PR ready for review, please make sure that:
<PR type>: <#issue number> <area>: <change being made>
fix: #1234 mempool: Introduce a cache for valid signatures
PR type
: fix, feat, INTERFACE BREAKING CHANGE, CONSENSUS BREAKING, build, chore, ci, docs, misc, perf, refactor, revert, style, testarea
: venus, venus-messager, venus-miner, venus-gateway, venus-auth, venus-market, venus-sealer, venus-wallet, venus-cluster, api, chain, state, vm, data transfer, mempool, message, block production, multisig, networking, paychan, proving, sealing, wallet