quackduck
Made commands harder to trigger.
Commands that are only a normal English word are harder to trigger in order not to trash unwillingly the chat in a normal conversation.
Maybe we should consider adding a noArgs flag to the command struct, since this may be enabled for a lot of commands.
Edit: or perhaps a maxArgs number limit!
Maybe we should consider adding a
noArgsflag to the command struct, since this may be enabled for a lot of commands.Edit: or perhaps a
maxArgsnumber limit!
Where would you want to add that definition? In the list of commands?
Where would you want to add that definition? In the list of commands?
Yeah, like a new field in cmd that gets defined for each command in the slices.
Where would you want to add that definition? In the list of commands?
Yeah, like a new field in
cmdthat gets defined for each command in the slices.
Sorry for forgetting to reply. I did something similar in the branch please_gocyclio but you didn't like it. Are you sure we should set it up for this issue?
Turns out we don't need a new field in that struct. The argsInfo fields already tells if we need arguments or not.
I'm not sure of this approach because a lot of commands check for when they don't receive any arguments.