Closed Dingying0410 closed 3 years ago
Merging #91 (9721ace) into main (8f9fc46) will increase coverage by
0.31%
. The diff coverage is100.00%
.:exclamation: Current head 9721ace differs from pull request most recent head 788afb6. Consider uploading reports for the commit 788afb6 to get more accurate results
@@ Coverage Diff @@
## main #91 +/- ##
==========================================
+ Coverage 97.67% 97.98% +0.31%
==========================================
Files 40 40
Lines 2367 2434 +67
==========================================
+ Hits 2312 2385 +73
+ Misses 55 49 -6
Impacted Files | Coverage Δ | |
---|---|---|
lib/redis_memo/future.rb | 98.46% <100.00%> (+0.10%) |
:arrow_up: |
lib/redis_memo/memoize_query/cached_select.rb | 93.89% <100.00%> (ø) |
|
lib/redis_memo/middleware.rb | 100.00% <100.00%> (+60.00%) |
:arrow_up: |
lib/redis_memo/options.rb | 88.13% <100.00%> (ø) |
|
spec/options_spec.rb | 100.00% <100.00%> (ø) |
|
spec/railtie_spec.rb | 100.00% <100.00%> (ø) |
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 8f9fc46...788afb6. Read the comment docs.
Summary
Adding rdoc for cache select. This is what I thought was useful when developing the features of #63, as it helps to understand the logic of parsing bind params, though this is not a file that will be used much by the users.