Skip to content

Remove SuiteSparse from dependencies#149

Merged
lostella merged 3 commits into
JuliaFirstOrder:masterfrom
lostella:fix-deps
Jan 15, 2026
Merged

Remove SuiteSparse from dependencies#149
lostella merged 3 commits into
JuliaFirstOrder:masterfrom
lostella:fix-deps

Conversation

@lostella

@lostella lostella commented Oct 27, 2023

Copy link
Copy Markdown
Member

Suite Sparse is been merged into SparseArrays.

  • Removing dependency on SuiteSparse in favor of SparseArrays
  • Bump minimum Julia requirement to 1.9
  • Adjust GH workflows accordingly
  • Update actions in GH workflows
  • Bump package version to 0.17

@codecov

codecov Bot commented Oct 27, 2023

Copy link
Copy Markdown

Codecov Report

Merging #149 (098d95e) into master (86dcef2) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

❗ Current head 098d95e differs from pull request most recent head 21d7f3d. Consider uploading reports for the commit 21d7f3d to get more accurate results

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   95.61%   95.52%   -0.09%     
==========================================
  Files          79       79              
  Lines        2483     2483              
==========================================
- Hits         2374     2372       -2     
- Misses        109      111       +2     
Files Coverage Δ
src/calculus/epicomposeQuadratic.jl 100.00% <ø> (ø)
src/functions/indAffine.jl 90.00% <ø> (ø)
src/functions/indAffineDirect.jl 97.50% <100.00%> (ø)
src/functions/indGraphSparse.jl 100.00% <ø> (ø)
src/functions/leastSquaresDirect.jl 93.42% <100.00%> (ø)
src/functions/quadraticDirect.jl 97.82% <100.00%> (ø)

... and 1 file with indirect coverage changes

@lostella

lostella commented Nov 4, 2023

Copy link
Copy Markdown
Member Author

This is apparently still a separate stdlib in Julia 1.6

@matbesancon

Copy link
Copy Markdown

@lostella with 1.10 being LTS this can probably be picked up again

@hakkelt

hakkelt commented Jan 15, 2026

Copy link
Copy Markdown
Contributor

I checked out this branch locally, rebased on the current master, and ran tests. There were no conflicts during the rebase, and all tests passed. @lostella, could you please merge this PR?

@lostella lostella merged commit 383d978 into JuliaFirstOrder:master Jan 15, 2026
7 of 8 checks passed
@hakkelt

hakkelt commented Jan 15, 2026

Copy link
Copy Markdown
Contributor

thanks!

@lostella lostella deleted the fix-deps branch January 16, 2026 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants