Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Samples for redis + jedis #240

Merged
merged 38 commits into from
Nov 3, 2023
Merged

Samples for redis + jedis #240

merged 38 commits into from
Nov 3, 2023

Conversation

yuanyuancin
Copy link
Collaborator

Motivation

Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.

Modification

Describe the idea and modifications you've done.

Result

Resolved or fixed #.

If there is no issue then describe the changes introduced by this PR.

Copy link

codecov bot commented Nov 2, 2023

Codecov Report

Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (ca9a253) 76.00% compared to head (b231480) 67.67%.
Report is 19 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #240      +/-   ##
============================================
- Coverage     76.00%   67.67%   -8.33%     
- Complexity        0      296     +296     
============================================
  Files            15       75      +60     
  Lines          1421     1680     +259     
  Branches          0      108     +108     
============================================
+ Hits           1080     1137      +57     
- Misses          243      470     +227     
+ Partials         98       73      -25     
Files Coverage Δ
...s/common/spring/ServerlessApplicationListener.java 83.33% <83.33%> (ø)
...verless/common/spring/MasterBizPropertySource.java 66.66% <66.66%> (ø)
...mon/spring/ServerlessEnvironmentPostProcessor.java 80.00% <80.00%> (ø)

... and 87 files with indirect coverage changes

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

Copy link
Collaborator

@lvjing2 lvjing2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yuanyuancin yuanyuancin merged commit ce930ef into master Nov 3, 2023
4 of 5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants