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
Fix #4541: Add again uid and mail options for user commands #4542
Conversation
This would be nice! |
As a workaround, you can do things like this:
|
We discussed the above, and one issue we have is that it's not very discoverable for junior developers or those unfamiliar with the shell. Making it an argument exposes it in the help docs, where the above requires understanding of shell string substitution. @weitzman I see you closed the related issue. Is there anything you would like to see in order to get a feature like this merged in? |
In general, this looks great. I am looking at our more strict validation in user:login and wondering if the warnings proposed here are sufficient. See drush/src/Commands/core/LoginCommands.php Lines 55 to 70 in 2cb8999
|
Sorry for the notification noise while I coax CircleCI run its tests on this PR. |
@weitzman sorry to bother you, but could this be backported to 10.x as well since it's still a stable release? Seems to be mergeable without any changes. |
Fix #4541: Add again uid and mail options for user command.