-
Notifications
You must be signed in to change notification settings - Fork 485
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
Deprecation fixes #643
Deprecation fixes #643
Conversation
717166d
to
31044e5
Compare
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.
Looks great, thanks for doing this! Will merge once Compat is out and the tests pass.
@@ -80,7 +80,7 @@ meta(m) = Docs.meta(m) | |||
nameof(x::Function) = typeof(x).name.mt.name | |||
nameof(b::Base.Docs.Binding) = b.var | |||
nameof(x::DataType) = x.name.name | |||
nameof(m::Module) = module_name(m) | |||
nameof(m::Module) = Compat.nameof(m) |
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.
I guess this could actually just extend the nameof
in Base. But that is beyond this PR.
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.
Yeah, we should actually remove the nameof
definitions here to avoid shadowing Base on 0.7.
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.
I did extend at first but that a bit weird. Either we leave it as is here, or upstream the other nameof
methods to base. Not sure how useful the other methods are though, so I think its fine to leave this as is.
@@ -214,8 +214,6 @@ end | |||
find_object(::Union{Function, DataType}, binding, ::Union{Union,Type{Union{}}}) = Utilities.Object(binding, Union{}) | |||
find_object(other, binding, typesig) = Utilities.Object(binding, typesig) | |||
|
|||
_method_exists(f, t) = method_exists(f, t) |
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.
Apparently this actually made sense back when Documenter still supported 0.4. Good to get rid of this. 😆
31044e5
to
69acbff
Compare
69acbff
to
55dcbc2
Compare
The Windows failures on 0.7 are unrelated. (They've been happening everywhere.) It'll need an admin to merge since it's a required check. |
Requires JuliaLang/Compat.jl#492, JuliaLang/Compat.jl#493
and JuliaLang/Compat.jl#494and then a new Compat release to pass on 0.6.