search

astrofrog / mpl-scatter-density

:zap: Fast scatter density plots for Matplotlib :zap:
BSD 2-Clause "Simplified" License
497 stars 25 forks source link

remove distutils function and change packaging.version #37

Closed ifurther closed 1 year ago

ifurther commented 1 year ago

distutils is removed in new version python

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.05% :warning:

Comparison is base (a890fa4) 88.43% compared to head (2a8c2bf) 88.38%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #37 +/- ## ========================================== - Coverage 88.43% 88.38% -0.05% ========================================== Files 11 11 Lines 536 534 -2 ========================================== - Hits 474 472 -2 Misses 62 62 ``` | [Files Changed](https://app.codecov.io/gh/astrofrog/mpl-scatter-density/pull/37?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Thomas+Robitaille) | Coverage Δ | | |---|---|---| | [mpl\_scatter\_density/tests/\_\_init\_\_.py](https://app.codecov.io/gh/astrofrog/mpl-scatter-density/pull/37?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Thomas+Robitaille#diff-bXBsX3NjYXR0ZXJfZGVuc2l0eS90ZXN0cy9fX2luaXRfXy5weQ==) | `100.00% <100.00%> (ø)` | |

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

astrofrog commented 1 year ago

Thanks for the PR! I have rebased this now that I've fixed the CI infrastructure.

astrofrog commented 1 year ago

We no longer need to use setup.py and we also don't need to check for the version in the tests anymore so actually simplifying this by removing the deprecated code.