Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ready for julia 1.10 #11

Closed
wants to merge 4 commits into from
Closed

ready for julia 1.10 #11

wants to merge 4 commits into from

Conversation

thorek1
Copy link
Contributor

@thorek1 thorek1 commented Sep 24, 2023

SparseArrays doesnt have AdjOrTransDenseMatrix anymore in julia 1.10

here is a fix to get it to work

@thorek1
Copy link
Contributor Author

thorek1 commented Nov 4, 2023

kind reminder: Julia 1.10 is coming closer

it would be nice if you could look into this

your package is a downward compatibility of my package and prevents it from passing tests

@thorek1 thorek1 mentioned this pull request Nov 4, 2023
@rasmushenningsson
Copy link
Collaborator

@jagot Do you have time to look at this? I'm currently on parental leave with very limit time on my hands. :)

@jagot
Copy link
Owner

jagot commented Dec 14, 2023

Sorry for missing this PR, I will try to take a look at it during the weekend.

@jagot jagot mentioned this pull request Dec 27, 2023
@jagot
Copy link
Owner

jagot commented Dec 27, 2023

Merged in #13, thank you! I wanted to revert parts of the CI config, since if we declare compatibility with Julia 1.4, we better test on that version too.

@jagot jagot closed this Dec 27, 2023
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