BedrockStreaming/RedisMock

Adding multiple members with SADD

Closed this issue 9 years ago · 3 comments

dchancogne commented 10 years ago

Any particular reason you do not support adding multiple members with SADD? Or just didn't get to it yet?

fdubost commented 10 years ago

No, there is no particular reason, just not enough time to add it. Feel free to implement it 😉

mikaelrandy commented 9 years ago

This issue is no more relevant, as SADD handle multiple members adding
https://github.com/M6Web/RedisMock/blob/master/src/M6Web/Component/RedisMock/RedisMock.php#L237

dchancogne commented 9 years ago

This is great. Thank you @mikaelrandy !