Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Require a space between the command and its arguments (if it has any) #123

Open
connorkuehl opened this issue Aug 6, 2022 · 0 comments
Open

Comments

@connorkuehl
Copy link
Owner

connorkuehl commented Aug 6, 2022

This will require some changes to the regexps used to determine if a message is a command.

This might also ripple out to some test cases in router_test.go since some of the assertions were made assuming that the space isn't variable width.

If a command has optional args then a space is not required unless they are present.

command requires args has optional args
announce ✔️
karma ✔️
top ✔️
bot ✔️
@connorkuehl connorkuehl added the has automated acceptance The issue is complete if its corresponding unit tests are passing label Aug 6, 2022
@connorkuehl connorkuehl removed the has automated acceptance The issue is complete if its corresponding unit tests are passing label Oct 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant