-
-
Notifications
You must be signed in to change notification settings - Fork 77
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 Symbolics to 6, (keep existing compat) #1016
CompatHelper: bump compat for Symbolics to 6, (keep existing compat) #1016
Conversation
3cc93f8
to
18bfcc1
Compare
@ChrisRackauckas is there a migration guide for what has changed / needs to be updated? |
If you're not using |
We use This use seems like it might be ordering dependent: Catalyst.jl/src/registered_functions.jl Line 143 in 36c10b8
|
In the Latex recipe I used |
That shouldn't be changed. The sorting comes down to whether it should lexicographically sort things that are stored as unsorted, which is commutative operations (add, mul). |
…56-903-00340686961
Awesome, success. |
Now for the docs. |
This pull request changes the compat entry for the
Symbolics
package from5.30.1
to5.30.1, 6
.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.