-
Notifications
You must be signed in to change notification settings - Fork 367
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
Use nonmissingtype instead of Missings.T #1941
Conversation
The latter is deprecated since Julia now exports nonmissingtype. Missings defines nonmissingtype on previous Julia versions. Also simplify the code a bit: there's no need to check T >: Missing before calling nonmissingtype, as it is the identity for other types.
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.
My only question is if nonmissingtype
is exported from Missings.jl for on Julia earlier than 1.3 (where it is in Base)
Failure is due to a deprecation warning printed by CategoricalArrays in a log test. JuliaData/CategoricalArrays.jl#205 should fix this. |
OK, but then we should first release CategoricalArrays.jl and then change the Project.toml in this PR to require its proper version also. Right? |
I don't think we need to require that version: the test failure doesn't reflect a real issue, it's just due to how we test that warnings are printed. |
OK |
CategoricalArrays tagged: JuliaRegistries/General#3292 |
Looks good to be merged. |
The latter is deprecated since Julia now exports nonmissingtype. Missings defines nonmissingtype on previous Julia versions. Also simplify the code a bit: there's no need to check T >: Missing before calling nonmissingtype, as it is the identity for other types.
The latter is deprecated since Julia now exports
nonmissingtype
.Missings defines
nonmissingtype
on previous Julia versions.Also simplify the code a bit: there's no need to check
T >: Missing
before callingnonmissingtype
, as it is the identity for other types.Waiting for JuliaRegistries/General#3224.