RedisTimeSeries / NRedisTimeSeries

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

Fix lgtm report #57

Closed AvitalFineRedis closed 3 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #57 (2eaabe1) into master (17d56ca) will increase coverage by 0.02%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
+ Coverage   95.03%   95.05%   +0.02%     
==========================================
  Files          14       14              
  Lines         604      607       +3     
==========================================
+ Hits          574      577       +3     
  Misses         30       30              
Impacted Files Coverage Δ
...RedisTimeSeries/DataTypes/TimeSeriesInformation.cs 100.00% <ø> (ø)
NRedisTimeSeries/DataTypes/TimeSeriesLabel.cs 100.00% <100.00%> (ø)
NRedisTimeSeries/DataTypes/TimeSeriesRule.cs 100.00% <100.00%> (ø)
NRedisTimeSeries/DataTypes/TimeSeriesTuple.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 17d56ca...2eaabe1. Read the comment docs.

lgtm-com[bot] commented 3 years ago

This pull request introduces 5 alerts and fixes 44 when merging 66e15cdc011743543c457998758fcfb7d90f7ff6 into 17d56ca250710063b2b3862777b6121ccc8da05b - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request fixes 44 alerts when merging e10bd0263fbcd21eccd6876dae3956cb111afda6 into 17d56ca250710063b2b3862777b6121ccc8da05b - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request fixes 44 alerts when merging 5ef5c3bd0d1a3fd5370daa1fd508b11e6eaafa42 into 17d56ca250710063b2b3862777b6121ccc8da05b - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 28 alerts and fixes 44 when merging 414e6dbae66fa57410e4b39409ef47844e457753 into 17d56ca250710063b2b3862777b6121ccc8da05b - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request fixes 44 alerts when merging 2eaabe14c5587edf1a22ae335aa25373d25d9f97 into 17d56ca250710063b2b3862777b6121ccc8da05b - view on LGTM.com

fixed alerts: