You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm fairly sure this has been discussed before but I can't find the issue.
The problem with your suggestion is that you won't be able to edit code blocks anymore since the REPL is line-oriented. It's not a text editor where you can move up and down. (Not counting ed.)
@bnoordhuis can't we trim the newlines from the code block to make sure that the block shows up on a single line? That's what the mongoDB REPL does, and is quite convenient.
Agree that we won't be able to edit it, but it would make this easier to tracking what was done, maybe we need to copy the block or we are demoing something on REPL...
This got fixed : #22153 and landed in master. Was surprised, I didn't notice this issue before (I saw the problem myself and fixed it :)). We can close this now.
Activity
bnoordhuis commentedon May 18, 2018
I'm fairly sure this has been discussed before but I can't find the issue.
The problem with your suggestion is that you won't be able to edit code blocks anymore since the REPL is line-oriented. It's not a text editor where you can move up and down. (Not counting
ed.)AyushG3112 commentedon May 18, 2018
@bnoordhuis can't we trim the newlines from the code block to make sure that the block shows up on a single line? That's what the mongoDB REPL does, and is quite convenient.
targos commentedon May 18, 2018
Shells like bash or zsh can do that:
https://asciinema.org/a/lBGxcTTxXoGGhPmhARMAt0CsG
Is it an option for Node?
bnoordhuis commentedon May 18, 2018
@AyushG3112 I don't think so. ASI means JS code with and without newlines doesn't necessarily behave the same.
hemanth commentedon May 20, 2018
Agree that we won't be able to edit it, but it would make this easier to tracking what was done, maybe we need to copy the block or we are demoing something on REPL...
antsmartian commentedon Nov 4, 2018
This got fixed : #22153 and landed in master. Was surprised, I didn't notice this issue before (I saw the problem myself and fixed it :)). We can close this now.
hemanth commentedon Nov 4, 2018
@antsmartian Awesome! We can try this on nightly, I guess.