-
Notifications
You must be signed in to change notification settings - Fork 37
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
Add blockchain.getBlockStats
, blockchain.getBestBlockHash
RPC
#256
Conversation
✔️ Deploy Preview for jellyfish-defi ready! 🔨 Explore the source changes: 26eeb71 🔍 Inspect the deploy log: https://app.netlify.com/sites/jellyfish-defi/deploys/60af24f3a777e20007e006d6 😎 Browse the preview: https://deploy-preview-256--jellyfish-defi.netlify.app |
@siradji: There are no 'kind' label on this PR. You need a 'kind' label to generate the release automatically.
DetailsI am a bot created to help the DeFiCh developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the DeFiCh/oss-governance-bot repository. |
size-limit report 📦
|
Codecov Report
@@ Coverage Diff @@
## main #256 +/- ##
==========================================
+ Coverage 96.49% 96.92% +0.42%
==========================================
Files 85 89 +4
Lines 2228 2408 +180
Branches 287 308 +21
==========================================
+ Hits 2150 2334 +184
+ Misses 78 74 -4
Continue to review full report at Codecov.
|
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.
please format your code properly, I am going to rebase your code from main. There was an issue with CI not failing due to lint issue.
packages/jellyfish-api-core/__tests__/category/blockchain.test.ts
Outdated
Show resolved
Hide resolved
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.
please check your PR before you open them. and rebase main, your code coverage is not accurate.
packages/jellyfish-api-core/__tests__/category/blockchain.test.ts
Outdated
Show resolved
Hide resolved
packages/jellyfish-api-core/__tests__/category/blockchain.test.ts
Outdated
Show resolved
Hide resolved
06c0c98
to
2f1e637
Compare
49e2a3f
to
2df1419
Compare
Code Climate has analyzed commit 26eeb71 and detected 0 issues on this pull request. View more on Code Climate. |
blockchain.getBlockStats
, blockchain.getBestBlockHash
RPC
packages/jellyfish-api-core/__tests__/category/blockchain.test.ts
Outdated
Show resolved
Hide resolved
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.
Pull request was converted to draft
* getBlockStats, getBestBlockHash RCP implementation * Fixed and implemented requested changes. * Fixed requested changes * Minor fixes * Changed assertion to toStricEquals, additional test for getblockstats * Update packages/jellyfish-api-core/src/category/blockchain.ts Co-authored-by: Fuxing Loh <[email protected]> * Update packages/jellyfish-api-core/__tests__/category/blockchain.test.ts Co-authored-by: Fuxing Loh <[email protected]> * Update packages/jellyfish-api-core/src/category/blockchain.ts Co-authored-by: Fuxing Loh <[email protected]> * Additional test cases for getBlockStats * Minor fixes * Update blockchain.md * Minor fixes * Update blockchain.md Co-authored-by: Fuxing Loh <[email protected]>
* updated to latest image * Updated to 1.7.0 rc5 tag * Fix mining test * Initial setup * Updated test * Updated EunosHeight in RegTest container * Added override pair * Tests passing, added docs * Increased code coverage * Add `blockchain.getBlockStats`, `blockchain.getBestBlockHash` RPC (#256) * getBlockStats, getBestBlockHash RCP implementation * Fixed and implemented requested changes. * Fixed requested changes * Minor fixes * Changed assertion to toStricEquals, additional test for getblockstats * Update packages/jellyfish-api-core/src/category/blockchain.ts Co-authored-by: Fuxing Loh <[email protected]> * Update packages/jellyfish-api-core/__tests__/category/blockchain.test.ts Co-authored-by: Fuxing Loh <[email protected]> * Update packages/jellyfish-api-core/src/category/blockchain.ts Co-authored-by: Fuxing Loh <[email protected]> * Additional test cases for getBlockStats * Minor fixes * Update blockchain.md * Minor fixes * Update blockchain.md Co-authored-by: Fuxing Loh <[email protected]> * Added remove oracle and tests * Fix test * Tidy test * Apply suggestions from code review Co-authored-by: Fuxing Loh <[email protected]> Co-authored-by: Fuxing Loh <[email protected]> Co-authored-by: Suraj Auwal <[email protected]> Co-authored-by: Fuxing Loh <[email protected]>
What kind of PR is this?:
/kind Feature
What this PR does / why we need it:
implementation of RCP #48
Which issue(s) does this PR fixes?:
Fixes #
Additional comments?: