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

Code cell #72

Closed
wants to merge 3 commits into from
Closed

Code cell #72

wants to merge 3 commits into from

Conversation

norci
Copy link
Contributor

@norci norci commented Dec 17, 2021

What about display the eval_tmpfile result in the REPL? so the behavior is similar to vs-code's Julia extension?

@gcv
Copy link
Owner

gcv commented Dec 18, 2021

An interesting idea, but needs refinement. As implemented, it typically prints the eval output immediately after the julia> prompt, which means I can't always look at the REPL and distinguish between input I typed and output generated by evals. At least, it needs to print the output on a new line, and it should try to highlight that it's eval output (maybe with colors, maybe by inserting # comments that clarify that they're output).

Related thought: CIDER uses overlays to display eval results in the source buffer. Maybe it makes sense to clone this feature.

Also, unless it becomes much more unobtrusive, this feature should be opt-in.

@gcv gcv mentioned this pull request Apr 14, 2022
@gcv
Copy link
Owner

gcv commented Apr 19, 2022

I just implemented this feature as an opt-in in f64e0d7: set to julia-snail-repl-display-eval-results to t to try it out.

@gcv gcv closed this Apr 19, 2022
@norci norci deleted the code-cell branch August 9, 2022 01:52
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 this pull request may close these issues.

2 participants