Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Helper methods to use a Jedis object within a JedisPool #3603

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

oscar-besga-panel
Copy link
Contributor

Add two helper methods to JedisPool

  • one that uses a lamda to operate with a jedis connection from the pool and doesn't return anything
  • other similar but it returns something

@codecov-commenter
Copy link

codecov-commenter commented Dec 9, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.68%. Comparing base (8b4f421) to head (5090680).

❗ Current head 5090680 differs from pull request most recent head fdcae99. Consider uploading reports for the commit fdcae99 to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3603      +/-   ##
============================================
+ Coverage     79.63%   79.68%   +0.05%     
- Complexity     5572     5575       +3     
============================================
  Files           301      301              
  Lines         15086    15091       +5     
  Branches       1134     1134              
============================================
+ Hits          12013    12026      +13     
+ Misses         2572     2565       -7     
+ Partials        501      500       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sazzad16 sazzad16 modified the milestone: 5.2.0 Mar 22, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants