Azure / azure-kusto-java

Microsoft Azure Kusto Library for Java
MIT License
38 stars 43 forks source link

Dont use timer if closed #312

Closed ohadbitt closed 1 year ago

github-actions[bot] commented 1 year ago

Test Results

269 tests  ±0   264 :heavy_check_mark: ±0   1m 57s :stopwatch: -9s   22 suites ±0       5 :zzz: ±0    22 files   ±0       0 :x: ±0 

Results for commit 4ba6966b. ± Comparison against base commit e2b288d1.

:recycle: This comment has been updated with latest results.

codecov[bot] commented 1 year ago

Codecov Report

Merging #312 (4ba6966) into master (e2b288d) will decrease coverage by 0.03%. The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master     #312      +/-   ##
============================================
- Coverage     56.50%   56.48%   -0.03%     
  Complexity      731      731              
============================================
  Files           106      106              
  Lines          3837     3842       +5     
  Branches        393      395       +2     
============================================
+ Hits           2168     2170       +2     
- Misses         1488     1489       +1     
- Partials        181      183       +2     
Files Changed Coverage Δ
...re/kusto/data/auth/PublicAppTokenProviderBase.java 53.33% <0.00%> (-3.81%) :arrow_down:
.../microsoft/azure/kusto/ingest/ResourceManager.java 76.30% <75.00%> (-0.51%) :arrow_down:
yogilad commented 1 year ago

Can you please add a description? This is also so the change list for the version can be easily composed