-
-
Notifications
You must be signed in to change notification settings - Fork 117
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
Changes invokelatest eval to run in current_module() #359
Conversation
test/runtests.jl
Outdated
@@ -1870,6 +1870,8 @@ let | |||
@test_throws MethodError Dates.Month(1) < Dates.Day(1) | |||
end | |||
|
|||
@test Compat.invokelatest(current_module) != Compat |
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.
=== current_module()
?
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, that's better
14e43e3
to
e22fae6
Compare
doh, my fault, sorry |
worth a patch release if you're in a hurry to use this in ijulia |
Okay, will tag a patch release. |
Tagged 0.25.1 (JuliaLang/METADATA.jl#9334) |
* `zeros` and `ones` with interface of `similar` (from #330) * `convert` between `Set` types (from #342) * `isassigned(::RefValue)` (from #345) * `unsafe_trunc(::Type{<:Integer}, ::Integer)` (from #344) * `bswap` for complex numbers (from #346) * Compat.StringVector (from #348) * `invokelatest` (from #352 and #359) * Misc. pre-0.6-only code * obsolete README enries
invokelatest
runs in module Compat on julia 0.5, so this:returns
Compat
in julia 0.5, but on 0.6-rc1.0 (which hasBase.invokelatest
defined) it returnsMain
.After this PR they both return
Main
.