Skip to content
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

varinfo killed #42143

Closed
PallHaraldsson opened this issue Sep 7, 2021 · 2 comments
Closed

varinfo killed #42143

PallHaraldsson opened this issue Sep 7, 2021 · 2 comments

Comments

@PallHaraldsson
Copy link
Contributor

a)

julia> using MLStyle

julia> varinfo(MLStyle, all=true, recursive=true, imported=true);
Killed

I'm not sure why, I'm on 1.7.0-beta3.0, and just discovered this function (that seems to work otherwise; with other packages).

b)
Is there a reason only Regex allowed?

It seemed better to allow Strings too (or intentional to disallow?), but I'm not sure why this doesn't work (otherwise could be a PR):

julia> import InteractiveUtils.varinfo

julia> varinfo(m::Module=Main, pattern::String=""; all::Bool = false, imported::Bool = false, sortby::Symbol = :name, recursive::Bool = false) = varinfo(m, Regex(pattern), all, imported, sortby, recursive)
varinfo (generic function with 5 methods)

julia> varinfo("m")
ERROR: MethodError: no method matching varinfo(::String)
@jakobnissen
Copy link
Contributor

Possible dup of #42045 ?

@KristofferC
Copy link
Member

Confirmed fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants