InjectiveLabs / sdk-python

Injective Exchange API for Python clients
Apache License 2.0
41 stars 27 forks source link

Feat/add chain streams support #243

Closed aarmoa closed 11 months ago

aarmoa commented 1 year ago
codecov[bot] commented 1 year ago

Codecov Report

Attention: 29 lines in your changes are missing coverage. Please review.

Comparison is base (cb38b09) 56.66% compared to head (1f053da) 56.59%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #243 +/- ## ========================================== - Coverage 56.66% 56.59% -0.07% ========================================== Files 17 17 Lines 2042 2071 +29 Branches 241 240 -1 ========================================== + Hits 1157 1172 +15 - Misses 848 862 +14 Partials 37 37 ``` | [Files](https://app.codecov.io/gh/InjectiveLabs/sdk-python/pull/243?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=InjectiveLabs) | Coverage Δ | | |---|---|---| | [pyinjective/async\_client.py](https://app.codecov.io/gh/InjectiveLabs/sdk-python/pull/243?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=InjectiveLabs#diff-cHlpbmplY3RpdmUvYXN5bmNfY2xpZW50LnB5) | `50.90% <66.66%> (+0.18%)` | :arrow_up: | | [pyinjective/core/network.py](https://app.codecov.io/gh/InjectiveLabs/sdk-python/pull/243?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=InjectiveLabs#diff-cHlpbmplY3RpdmUvY29yZS9uZXR3b3JrLnB5) | `42.62% <18.18%> (+1.98%)` | :arrow_up: | | [pyinjective/composer.py](https://app.codecov.io/gh/InjectiveLabs/sdk-python/pull/243?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=InjectiveLabs#diff-cHlpbmplY3RpdmUvY29tcG9zZXIucHk=) | `50.28% <41.37%> (-1.43%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

aarmoa commented 1 year ago

The code coverage in the PR decreases a little bit. But it will be increased in the PR we are preparing to add the low level components in the SDK that the TS SDK has

aarmoa commented 1 year ago

Changes are ready to be merged. I am just waiting for a confirmation on the chain stream endpoints for the different networks