search

nim-lang / redis

Official redis wrapper for Nim.
Other
128 stars 35 forks source link

Async redis mixes up responses #6

Closed dom96 closed 6 years ago

dom96 commented 6 years ago

This is a pretty serious flaw. The tawaitorder.nim file (which I will soon commit) demonstrates the issue.

euantorano commented 6 years ago

Commit 1512b87 causes a CI failure:

Hint: used config file '/nim/config/nim.cfg' [Conf]
/usr/src/redis/src/redis.nim(138, 19) template/generic instantiation from here
/usr/src/redis/src/redis.nim(147, 22) template/generic instantiation from here
/usr/src/redis/src/redis.nim(112, 18) template/generic instantiation from here
/usr/src/redis/src/redis.nim(106, 36) Error: cannot instantiate: 'none[string]'
Exited with code 1

Probably should have submitted it as a PR rather than commiting straight to master as the build is now marked as failing.