make matching case-insensitive on zulip

[jdonszelmann]

I often use work show from my phone, which automatically capitalizes the first letter in my sentence. I see no reason why we would care about case in these commands, so just converting everything to lowercase shouldn't make a difference to most people while it makes the bot do the right thing more often.

[jdonszelmann]

let me know if this makes sense, I thought I wouldn't open an issue since it's such a small change, just making the changes and discussing it here made more sense to me :)

[Urgau]

I don't think we currently have any zulip commands that cares about it's argument capitalization, so this would be technically fine.

I still wonder if it wouldn't be better to only lowercase the very first letter of the command.

cc @Kobzol

[jdonszelmann]

We currently do not, though I just realized that commands that take parameters probably shouldn't get their parameter lowercased. So maybe we should also only do the first word in the arguments, or only do so after the unchanged version failed to parse

[Urgau]

So maybe we should also only do the first word in the arguments, or only do so after the unchanged version failed to parse

Both make sense to me. The first one would be simpler to implement.

[Kobzol]

I think that most of our parameters should already be case resistant. But wouldn't using this: https://docs.rs/clap/latest/clap/struct.Arg.html#method.ignore_case be less of a hack?

[Urgau]

I didn't knew about that setting, it indeed seems much better.

[Kobzol]

Sadly AFAIK it cannot be currently applied to the whole command, so we need to add it to each argument separately...

[jdonszelmann]

I looked for that but didn't find it earlier. Would be nice to apply globally ye...