search

BedrockStreaming / RedisMock

A simple PHP Redis mock
http://tech.m6web.fr/redismock-qui-a-bouchonne-mon-redis.html
MIT License
81 stars 60 forks source link

Return an integer reply for EXISTS #68

Closed kjnsn closed 6 years ago

kjnsn commented 6 years ago

In order to be consistent with the redis protocol: https://redis.io/commands/exists, the command should return an integer rather than a boolean. I have confirmed that predis follows the behaviour from the redis spec.

omansour commented 6 years ago

txs ! v3.3.2

omansour commented 6 years ago

damn !

2018-06-04 9:39 GMT+02:00 BOUYGE notifications@github.com:

@gbouyge approved this pull request.

Warning, this will introduce a BC break.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/M6Web/RedisMock/pull/68#pullrequestreview-125462354, or mute the thread https://github.com/notifications/unsubscribe-auth/ABFCWoqvzBBF1hXHxMfL6E41w_6WUzYxks5t5OQcgaJpZM4UYf27 .

-- Olivier Mansour http://oliviermansour.com/ http://fr.linkedin.com/in/omansour

omansour commented 6 years ago

v4.0.0