-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 psysh 'requireSemicolons' in php:cli #5160
Comments
Thought a possible alternative might be to change it dynamically from within the REPL itself, but it looks like that's not possible at the mo: bobthecow/psysh#361 |
Ah, I can use a psysh config file without any change to drush. In combination with the PSYSH_CONFIG=config-multiline.php drush php I'll leave the request open because it's not super convenient, see what others think (: Thanks! |
K, I'm going to close this. If you'd like to be able to copy and paste a single logical line that's spread over multiple actual lines in
Thanks! |
You should also be able to add a |
Is your feature request related to a problem? Please describe.
By default psysh does a kind of JS-like automatic semicolon insertion for convenience, automatically adding one after each line break. And a lot of time it's convenient. But I'm always frustrated when I have a bit of multiline code that I want to test in the REPL, and instead of just copying and pasting, I have to reformat it onto one line.
Describe the solution you'd like
Psysh has an option that requires semicolons:
requireSemicolons
. I've tested it locally and it works as advertised. I suggest supporting that. I'm not sure if it makes more sense to have a dedicated option, or a general mechanism to pass psysh config options through (insert obligatory xkcd reference!).Thanks for this life-saving tool ❤️
The text was updated successfully, but these errors were encountered: