-
Notifications
You must be signed in to change notification settings - Fork 29.6k
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
Support multiline input with user eval
function
#2939
Labels
feature request
Issues that request new features to be added to Node.js.
repl
Issues and PRs related to the REPL subsystem.
Comments
brendanashworth
added
repl
Issues and PRs related to the REPL subsystem.
feature request
Issues that request new features to be added to Node.js.
labels
Sep 18, 2015
Fix for multiline input when using custom eval function @ nodejs/node-v0.x-archive#25587 |
@blakeembrey Exporting recoverable might be a simpler solution to what I was proposing in my pull request and I think simple is good. I guess we need some input as to which would be preferred. I think it would be good to submit your pull request 👍 |
evanlucas
pushed a commit
that referenced
this issue
May 17, 2016
Allow REPL consumers to callback with a `Recoverable` error instance and trigger multi-line REPL prompts. Fixes: #2939 PR-URL: #3488 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]>
This was referenced Jun 29, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature request
Issues that request new features to be added to Node.js.
repl
Issues and PRs related to the REPL subsystem.
Currently, the node repl module has the capability to do multiline input when the error is an
instanceof Recoverable
. Unfortunately, the constructor is not exported from therepl
module which results in hacky ways to achieve a multi-line REPL experience like node. By exporting the error constructor, consumers could create their own recoverable instances to function identical to before and have multi-line input work naturally.References:
node/lib/repl.js
Line 413 in a69ab27
Edit: Mostly a question, if it's reasonable to add I can issue a PR for a one-line export change.
The text was updated successfully, but these errors were encountered: