-
Notifications
You must be signed in to change notification settings - Fork 1
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
Repl mode #222
Repl mode #222
Conversation
Preview should be up in a couple of mins at |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #222 +/- ##
==========================================
+ Coverage 83.42% 83.44% +0.01%
==========================================
Files 66 66
Lines 3385 3491 +106
==========================================
+ Hits 2824 2913 +89
- Misses 561 578 +17
☔ View full report in Codecov by Sentry. |
Preview should be up in a couple of mins at |
2 similar comments
Preview should be up in a couple of mins at |
Preview should be up in a couple of mins at |
… though they don't generate files
Preview should be up in a couple of mins at |
Preview should be up in a couple of mins at |
Preview should be up in a couple of mins at |
1 similar comment
Preview should be up in a couple of mins at |
Preview should be up in a couple of mins at |
Preview should be up in a couple of mins at |
Preview should be up in a couple of mins at |
1 similar comment
Preview should be up in a couple of mins at |
Preview should be up in a couple of mins at |
Preview should be up in a couple of mins at |
closes #123
closes #219
closes #226 (force re-eval if anything changes in the block, this should be the case even if we evaluate line by line)
moved the error test stuff to #230
Note: repl-repl julia error handling requires tweakingok doneeval_code_cell!
which callseval_code_cell
which callseval_nb_cell
,_process_eval_error
, a repl argument should be passed to allow the error, not warn, not throw, and show the trimmed error.