search

sorentwo / readthis

:newspaper: Pooled active support compliant caching with redis
MIT License
504 stars 40 forks source link

fix the guard for max amount of serializers #46

Closed juanperi closed 8 years ago

juanperi commented 8 years ago

While reviewing #45, I just spotted that the guard is wrong, as allows up until 8 serializers. At the same time, the test was wrong, as adding up until 10 serializers more doesn't ensure that we can have up to 7

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 68709fdef2f09f05b51a65f8d4a4e87abee1dad4 on epilgrim:fix/amount-of-serializers-guard into 553d45273d179ece53c723cb9b4a827a0c378d5b on sorentwo:master.

sorentwo commented 8 years ago

Thanks for catching these (and for helping with the rassoc fix in the other PR). Merged locally, closed with f336167.