Update ci.yml

[dlfivefifty]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 94.81%. Comparing base (81eee0d) to head (60a1a9a).

Files	Patch %	Lines
src/eigenGeneral.jl	0.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
- Coverage   95.04%   94.81%   -0.24%     
==========================================
  Files          11       11              
  Lines        1636     1640       +4     
==========================================
  Hits         1555     1555              
- Misses         81       85       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[andreasnoack]

There are many of the changes here I don't understand. I hadn't realized that lts had been disabled in #85 but, except for that one, why are the changes needed? It's deliberate that I don't test on Mac and Windows since there are no binary dependencies so it doesn't seem worth the extra CI time.

I can see that the lts tests actually reveal some issues. Current master seems to rely on some methods in Julia that don't exist in Julia 1.6.

Update: can see that you are working on fixing the test failures.

[dlfivefifty]

I just copied CI from another repository (I think FillArrays.jl) which @jishnub made changes to...

I can revert the ci.yml and just add a test for lts.

For reasons that are beyond my comprehension the tests on ubuntu/macos are using test files from a different branch which makes no sense to me so maybe reverting the CI will avoid this issue...

[dlfivefifty]

Oh sorry! I didn't see that you merged the other PR. That's why the tests are failing

[dlfivefifty]

I think we can ignore the codecov failure since it relates to 1.6 specific code

[andreasnoack]

Thanks for the fixes