Skip to content
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

commands are too easy to use accidentally #36

Closed
Tecuya opened this issue Dec 16, 2022 · 3 comments
Closed

commands are too easy to use accidentally #36

Tecuya opened this issue Dec 16, 2022 · 3 comments
Labels
enhancement New feature or request

Comments

@Tecuya
Copy link
Contributor

Tecuya commented Dec 16, 2022

If someone provides a prompt like:

new cars are safer than old cars right?

or

session tracking in php is easy, right?

It's going to accidentally be triggering the new command, and the session command.

But what's the solution? Do we make all commands require a fore slash or something? new becomes /new?

Does anyone have an opinion, is it worth changing?

@kylrth
Copy link

kylrth commented Dec 26, 2022

If you're seeking opinions, mine would be to prepend with a slash. Ambiguity bad

@mmabrouk
Copy link
Collaborator

I also agree with requiring slashes

@mmabrouk mmabrouk added the enhancement New feature or request label Jan 17, 2023
@thehunmonkgroup
Copy link
Collaborator

implemented long ago, command leader is forward slash now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants