RFC: `sismember` to return bool

[mkmkme]

At this moment, sismember returns an integer representing whether a value is a member of the set:

>>> v.smembers(56)
['0', '1', '2', '3', '4', '5', '6', '7', '8', '9']
>>> v.sismember(56, 1)
1
>>> v.sismember(56, 999)
0

Similarly, smismember returns a list of literals instead of bools:

>>> v.smismember(56, range(11), 9)
[1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 1]

I have a question to valkey-py users: would you benefit if these functions returned bool instead? So that True would be returned instead of 1 and False instead of 0.

I invite every valkey-py user of these functions to chime in.

[amirreza8002]

hi 👋
tho I'm not using this method anywhere, but it seems the logical thing would be to return a bool

tho in theory it wouldn't matter much either way,
cause the check:
if v.sismember(1, 2):
will work the same either way
unless someone is explicitly checking for int, which i don't see why

in django-valkey (following django-redis) we're casting it to bool before returning