Is zrange with bylex supported?

[kchonka]

Not sure if it's a bug yet, but I'm getting a redis.exceptions.ResponseError: min or max is not a float error when I try to use zrange with the bylex=True arg

[cunla]

Hi, I see you opened a bug for this. Would you like to fix it as you pretty much pinpointed the issue (zrange doesn't handle bylex param)?

[kchonka]

Yep, can do!

[cunla]

Great, thank you.

Some pointers:

• There is a version variable coming from one of the mixin you can use, so you can add a condition here for bylex and version (something like casematch(arg, b'byscore') and self.version>=7).
• Alternatively, you can scan the args in SortedSetCommandsMixin.zrange and call _zrangebylex if bylex is on.
• When writing tests for it, you can use @pytest.mark.min_server(...) and @pytest.mark.max_server(...).
  Please write tests to see that on redis6 bylex is raising an error and for redis7 bylex is behaving as expected.

I am happy to have another person digging into the small details which I might have missed. If you are interested in being more involved - please sign up here. After a few PRs, I will add you to the repo collaborators.