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

Pkg REPL mode broken in 1.11-rc4 #1119

Closed
nilshg opened this issue Sep 26, 2024 · 3 comments · Fixed by #1124
Closed

Pkg REPL mode broken in 1.11-rc4 #1119

nilshg opened this issue Sep 26, 2024 · 3 comments · Fixed by #1124

Comments

@nilshg
Copy link

nilshg commented Sep 26, 2024

Already noticed this in rc3 but it's still a problem and I've just set up a completely new machine so could confirm that this happens on a fresh install (Windows 11, Julia 1.11-rc4 installed through Juliaup, then just add IJulia; using IJulia; notebook() and allowing the standard install of miniconda).

Then in a notebook I do:

] st

and get:

MethodError: no method matching do_cmd(::IJulia.MiniREPL, ::String; do_rethrow::Bool)
The function `do_cmd` exists, but no method is defined for this combination of argument types.

Closest candidates are:
  do_cmd(::Pkg.REPLMode.Command, ::Any) got unsupported keyword argument "do_rethrow"
   @ Pkg ...\.julia\juliaup\julia-1.11.0-rc4+0.x64.w64.mingw32\share\julia\stdlib\v1.11\Pkg\src\REPLMode\REPLMode.jl:398

I assume this is related to the excision of Pkg?

@nilshg
Copy link
Author

nilshg commented Oct 8, 2024

Just to prevent things from getting slackholed, @mkitti identified that in this commit:

JuliaLang/Pkg.jl@12d2de1

the do_rethrow keyword of do_cmd got removed. He suggested potentially dropping the keyword at the call site:

escape_string(code[2:end]) * "\"; do_rethrow=true)"

I'm not familiar enough with the IJulia codebase to understand whether that's a good idea though!

@nilshg
Copy link
Author

nilshg commented Oct 18, 2024

This is great, thanks - might this warrant a patch release?

@stevengj
Copy link
Member

stevengj commented Nov 5, 2024

JuliaRegistries/General#118774

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

Successfully merging a pull request may close this issue.

2 participants