-
Notifications
You must be signed in to change notification settings - Fork 26
CompatHelper: bump compat for LinearSolve to 3, (keep existing compat) #387
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
CompatHelper: bump compat for LinearSolve to 3, (keep existing compat) #387
Conversation
b373092
to
975944e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #387 +/- ##
==========================================
- Coverage 93.59% 87.15% -6.45%
==========================================
Files 44 45 +1
Lines 2826 3152 +326
==========================================
+ Hits 2645 2747 +102
- Misses 181 405 +224 ☔ View full report in Codecov by Sentry. |
I think this dependency issue will be solved after the following PR is merged: |
Also need to wait for them to bump the new version of |
We first need SciML/LinearSolve.jl#573 to be resolved. |
The circular dependency issue has been solved. I also fixed the code quality issue directly in this PR (since it's a reasonable small change). |
This pull request changes the compat entry for the
LinearSolve
package from2
to2, 3
.This keeps the compat entries for earlier versions.
Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.