Build against multiple redis versions.

seomoz / qless-core

Core Lua Scripts for qless

MIT License

85 stars 33 forks source link

Build against multiple redis versions. #68

Closed bkirz closed 7 years ago

bkirz commented 7 years ago

This way, we can better address problems like #64 that are specific to certain redis versions. Once this is merged, we can rebase #64 on top of it and confirm that it fixes the issue before merging.

bkirz commented 7 years ago

@b4hand, after talking to @myronmarston about it, I'd like to:

Merge this with an allow_failures rule for redis 3.2.8.
Validate that #64 fixes the issue and merge it.
Submit a followup that removes the allow_failures rule.

How's that sound?