RedisTimeSeries / NRedisTimeSeries

.Net Client for RedisTimeSeries
https://redistimeseries.io
BSD 3-Clause "New" or "Revised" License
28 stars 11 forks source link

groupby_reduce #48

Closed AvitalFineRedis closed 3 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #48 (4af6a76) into master (9b17c1f) will decrease coverage by 1.33%. The diff coverage is 70.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
- Coverage   96.36%   95.03%   -1.34%     
==========================================
  Files          13       14       +1     
  Lines         578      604      +26     
==========================================
+ Hits          557      574      +17     
- Misses         21       30       +9     
Impacted Files Coverage Δ
NRedisTimeSeries/Extensions/ReduceExtensions.cs 35.71% <35.71%> (ø)
NRedisTimeSeries/Commands/CommandArgs.cs 100.00% <100.00%> (ø)
NRedisTimeSeries/TimeSeriesClient.cs 100.00% <100.00%> (ø)
NRedisTimeSeries/TimeSeriesClientAsync.cs 93.65% <100.00%> (ø)
NRedisTimeSeries/TimeSeriesClientAux.cs 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 9b17c1f...4af6a76. Read the comment docs.

lgtm-com[bot] commented 3 years ago

This pull request introduces 6 alerts when merging 27c5786036f715c87ba7649f24707057f4def757 into 9b17c1f88679ccf2d9eabf91582b2e1b3f648a06 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 6 alerts when merging 4af6a762674b7f0f83c36169928a76eb799d0932 into 9b17c1f88679ccf2d9eabf91582b2e1b3f648a06 - view on LGTM.com

new alerts: