Closed donaldong closed 3 years ago
Merging #93 (ac437f5) into main (91ec911) will not change coverage. The diff coverage is
n/a
.:exclamation: Current head ac437f5 differs from pull request most recent head b1983d7. Consider uploading reports for the commit b1983d7 to get more accurate results
@@ Coverage Diff @@
## main #93 +/- ##
=======================================
Coverage 97.98% 97.98%
=======================================
Files 40 40
Lines 2434 2434
=======================================
Hits 2385 2385
Misses 49 49
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 91ec911...b1983d7. Read the comment docs.
Just noticed we didn't say anything about the min Redis version in the README file. This adds it.
We can also check the min Redis version in runtime (raise an error if the Redis version is too old) but that might be an overkill assuming users would read the README page first. (and might be too strict, since it's still fine on old Redis versions if it does not run out of memory)