-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
REPL: show user loaded packages in the banner #58366
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
base: master
Are you sure you want to change the base?
Conversation
c06edd4
to
dae3222
Compare
dae3222
to
2a0d28c
Compare
loaded_modules_string = "" | ||
else | ||
loaded_modules_string = "Loaded packages: $(join(loaded_user_models, ", "))" | ||
end |
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.
What happens if I put OmniPackage's direct and indirect dependencies in my startup.jl
? Do you still like the outcome?
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. Good point. I think it's reasonable for this to just truncate with ...
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.
If the point is for people to know if Revise is loaded, why not have Revise modify the julia prompt to provide a small unobtrusive indicator, e.g. |
Julia + Revise
More seriously..
Or a different green? That's a bit too subtle perhaps. Maybe underline? A prefix like you suggest might be ok. We do that in Pkg already.. |
@Keno, that's a really good idea. |
My browser font was so small I didn't realize your suggestion was an arrow circle. Yeah I like that.
|
Wouldn't that break copy-pasting terminal output? Given how much Revise is popular, that'd cause inconvenience to many users. |
The REPL pasting code is robust to changes in the prompt (that's how copy/pasting non-standard REPL modes works). I don't know if it's robust to prefix changes outside of the Pkg mode, but I'm sure it could be adjusted. |
I guess we could safely do a color-only change. Say blue instead of green? |
Blue is Pkg mode. |
I think it would be helpful to inform the user what was automatically loaded by startup.jl (or by
-ie "using Foo"
but less useful)In particular it's helpful to be reminded whether Revise is loaded when working between multiple machines.