Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Simplify CI configuration with Redismod Docker #50

Merged
merged 2 commits into from
Jul 25, 2022
Merged

Conversation

dengliming
Copy link
Owner

closes #49

@codecov
Copy link

codecov bot commented Jul 25, 2022

Codecov Report

Merging #50 (8bb623f) into master (62b85d1) will decrease coverage by 3.22%.
The diff coverage is 5.26%.

@@             Coverage Diff              @@
##             master      #50      +/-   ##
============================================
- Coverage     65.82%   62.59%   -3.23%     
+ Complexity      635      627       -8     
============================================
  Files           111      116       +5     
  Lines          2250     2334      +84     
  Branches        186      190       +4     
============================================
- Hits           1481     1461      -20     
- Misses          665      769     +104     
  Partials        104      104              
Impacted Files Coverage Δ
...github/dengliming/redismodule/redisai/RedisAI.java 71.60% <0.00%> (-11.96%) ⬇️
...ming/redismodule/redisai/args/StoreScriptArgs.java 0.00% <0.00%> (ø)
...gliming/redismodule/redisai/protocol/Keywords.java 100.00% <100.00%> (ø)
...ng/redismodule/redisai/protocol/RedisCommands.java 100.00% <100.00%> (ø)
...module/redisai/protocol/decoder/ScriptDecoder.java 7.14% <0.00%> (-92.86%) ⬇️
...b/dengliming/redismodule/redisai/model/Script.java 0.00% <0.00%> (-87.50%) ⬇️
...ngliming/redismodule/common/util/TestSettings.java 66.66% <0.00%> (ø)
...b/dengliming/redismodule/common/util/ArgsUtil.java 0.00% <0.00%> (ø)
...engliming/redismodule/common/api/RCommonBatch.java 0.00% <0.00%> (ø)
... and 1 more

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 62b85d1...8bb623f. Read the comment docs.

test

test

Add support for AI.SCRIPTSTORE command

fix

fix

fix

fix

fix

fix

fix

cleanup
@dengliming dengliming merged commit af2277d into master Jul 25, 2022
@dengliming dengliming deleted the ci_build branch July 25, 2022 16:15
dengliming added a commit that referenced this pull request Aug 2, 2022
* Simplify CI configuration with Redismod Docker
* Add support for AI.SCRIPTSTORE command
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Simplify CI configuration with Redismod Docker
1 participant