-
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 listLatestRawPrices rpc #352
Conversation
� Conflicts: � packages/jellyfish-api-core/src/category/oracle.ts � website/docs/jellyfish/api/oracle.md
Code Climate has analyzed commit 0409334 and detected 0 issues on this pull request. View more on Code Climate. |
packages/jellyfish-api-core/__tests__/category/oracle/listLatestRawPrices.test.ts
Outdated
Show resolved
Hide resolved
packages/jellyfish-api-core/__tests__/category/oracle/listLatestRawPrices.test.ts
Outdated
Show resolved
Hide resolved
size-limit report 📦
|
Still doing. Haven't finished |
Codecov Report
@@ Coverage Diff @@
## main #352 +/- ##
==========================================
- Coverage 97.16% 97.13% -0.03%
==========================================
Files 91 91
Lines 2642 2652 +10
Branches 339 341 +2
==========================================
+ Hits 2567 2576 +9
- Misses 75 76 +1
Continue to review full report at Codecov.
|
✔️ Deploy Preview for jellyfish-defi ready! 🔨 Explore the source changes: ff4aab0 🔍 Inspect the deploy log: https://app.netlify.com/sites/jellyfish-defi/deploys/60c2f8512f4955000833b15c 😎 Browse the preview: https://deploy-preview-352--jellyfish-defi.netlify.app |
…ices # Conflicts: # packages/jellyfish-api-core/src/category/oracle.ts # website/docs/jellyfish/api/oracle.md
packages/jellyfish-api-core/__tests__/category/oracle/listLatestRawPrices.test.ts
Show resolved
Hide resolved
packages/jellyfish-api-core/__tests__/category/oracle/listLatestRawPrices.test.ts
Outdated
Show resolved
Hide resolved
packages/jellyfish-api-core/__tests__/category/oracle/listLatestRawPrices.test.ts
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.
A few improvements you should make, won't block your progress now. Tackle in another PR, make it a standalone PR, don't mix with the others.
What kind of PR is this?:
/kind feature
What this PR does / why we need it:
Add listLatestRawPrices rpc
Which issue(s) does this PR fixes?:
Fixes part of #202
Additional comments?: