-
Notifications
You must be signed in to change notification settings - Fork 5
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 SpecialFunctions to 2, (keep existing compat) #10
base: main
Are you sure you want to change the base?
CompatHelper: bump compat for SpecialFunctions to 2, (keep existing compat) #10
Conversation
@@ -23,7 +23,7 @@ FourierTools = "0.2.1" | |||
IndexFunArrays = "0.2" | |||
Interpolations = "0.12, 0.13" | |||
QuadGK = "2.4.1" | |||
SpecialFunctions = "1, 1.1, 1.2" | |||
SpecialFunctions = "1, 1.1, 1.2, 2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why there was 1, 1.1, 1.2
? That's equivalent to just 1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably because I didn't know what I was doing :D
Or maybe compat helper merged this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CompatHelper wouldn't do that since these aren't different major versions. For the record 9538ef1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, why is SpecialFunctions both in the main deps and in the extras? That's useless
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing that out.
I try to fix that once I have time re-designing the general implementation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wait, doesn't SpecialFunctions need to be mentioned in extras when I need it for testing purpose as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No.
This pull request changes the compat entry for the
SpecialFunctions
package from1, 1.1, 1.2
to1, 1.1, 1.2, 2
.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.