FillArrays.jl icon indicating copy to clipboard operation
FillArrays.jl copied to clipboard

Set julia compat to v1.10

Open jishnub opened this issue 1 year ago • 1 comments

Since v1.10 is the LTS now, we may focus development only on v1.10+. Versions of this pacakge <v1.15 would continue to be compatible with julia v1.6.

jishnub avatar Dec 02 '24 12:12 jishnub

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 99.48%. Comparing base (6f61dc3) to head (173acaf). :warning: Report is 14 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #397      +/-   ##
==========================================
- Coverage   99.91%   99.48%   -0.44%     
==========================================
  Files           8        8              
  Lines        1147     1154       +7     
==========================================
+ Hits         1146     1148       +2     
- Misses          1        6       +5     

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

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Dec 02 '24 13:12 codecov[bot]

I think it would be a good opportunity to also clean up existing VERSION checks in the package and the tests.

devmotion avatar Oct 02 '25 10:10 devmotion

It actually seems that Julia < 1.10.10 is completely untested at this point: https://github.com/JuliaArrays/FillArrays.jl/blob/f160c66ce092712e47ad726dee0e4212c77210d1/.github/workflows/ci.yml#L40-L43 You might want to test min as well (which currently would be 1.6.0 and with this PR 1.10.0, in contrast to lts which is 1.10.10).

devmotion avatar Oct 02 '25 10:10 devmotion

I think testing 1.10.0 in addition to 1.10.10 is excessive. If a bug is ever triggered only in 1.10.0 it's not a good use of anyones time to fix it.

dlfivefifty avatar Oct 05 '25 19:10 dlfivefifty

If a bug is ever triggered only in 1.10.0 it's not a good use of anyones time to fix it.

In principle, you don't have to fix anything apart from correcting the Julia compat entry: If the package does not support 1.10.0, it should not be declared to be compatible with 1.10.0.

Currently, the problem is even more sever as not testing min means that not only versions 1.10.0 - 1.10.9 but all of 1.6.0 - 1.10.9 are not tested. If you'd like to reduce CI times, IMO it would actually be better to skip pre and/or to test min and 1 instead of lts and 1 (based on the assumption that if e.g. tests pass with 1.10.0 they most likely also pass with 1.10.10).

devmotion avatar Oct 05 '25 20:10 devmotion

I agree that in theory we should be testing v1.6.x right now. But one can argue if a version of Julia is no longer supported then there shouldn't be an expectation that packages continue to work

I'm less worried about CI time than I am additional developer burden. Every extra test is an extra point of failure...

dlfivefifty avatar Oct 06 '25 08:10 dlfivefifty

can this be closed?

dlfivefifty avatar Oct 31 '25 21:10 dlfivefifty