highcharts-for-python / highcharts-core

Python wrapper for the Highcharts Core JavaScript library
https://www.highcharts.com/integrations/python
Other
58 stars 13 forks source link

Merging bugfix for #200 - `np.isnan()` handling #202

Closed hcpchris closed 2 months ago

hcpchris commented 2 months ago

Merging @ThomasGL's bugfix from #201 into develop as commented here.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 87.95%. Comparing base (b6925a1) to head (5bc9978). Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
highcharts_core/js_literal_functions.py 66.66% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #202 +/- ## =========================================== - Coverage 87.96% 87.95% -0.02% =========================================== Files 210 210 Lines 31164 31166 +2 Branches 2552 2553 +1 =========================================== - Hits 27413 27411 -2 - Misses 2791 2793 +2 - Partials 960 962 +2 ``` | [Flag](https://app.codecov.io/gh/highcharts-for-python/highcharts-core/pull/202/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=highcharts-for-python) | Coverage Δ | | |---|---|---| | [](https://app.codecov.io/gh/highcharts-for-python/highcharts-core/pull/202/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=highcharts-for-python) | `87.95% <66.66%> (?)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=highcharts-for-python#carryforward-flags-in-the-pull-request-comment) to find out more.

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