search

astropy / specutils

An Astropy coordinated package for astronomical spectroscopy. Maintainers: @rosteen @keflavich @eteq
http://specutils.readthedocs.io/en/latest/
166 stars 127 forks source link

TST: Fix devdeps and clean up matrix #1111

Closed pllim closed 10 months ago

pllim commented 10 months ago

Fix devdeps job and clean up the testing infrastructure in general (some outdated stuff and unnecessary stuff from copy-paste).

xref https://github.com/astropy/astropy/issues/15638

Close #1089

Fix #1036

codecov[bot] commented 10 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (029ddfd) 70.74% compared to head (b353f77) 70.80%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1111 +/- ## ========================================== + Coverage 70.74% 70.80% +0.06% ========================================== Files 64 64 Lines 4484 4484 ========================================== + Hits 3172 3175 +3 + Misses 1312 1309 -3 ```

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

pllim commented 10 months ago

I see numpy 1.x vs 2.x ABI warning and dateutil deprecation warning in the devdeps log here but somehow it did not fail pytest even though we are turning warnings into exceptions. I think maybe it is because in devdeps, we install quite a few things from source and some of those might be emitting these warnings, not us, so maybe it is okay to ignore as long as CI is green here.

rosteen commented 10 months ago

Thanks!