Skip to content

Ensure ParametersAreNonnullByDefault is present in all packages#2879

Merged
bogdandrutu merged 1 commit intoopen-telemetry:mainfrom
bogdandrutu:paramsnotnull
Feb 18, 2021
Merged

Ensure ParametersAreNonnullByDefault is present in all packages#2879
bogdandrutu merged 1 commit intoopen-telemetry:mainfrom
bogdandrutu:paramsnotnull

Conversation

@bogdandrutu
Copy link
Copy Markdown
Member

Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 18, 2021

Codecov Report

Merging #2879 (7c06c65) into main (59bf511) will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #2879      +/-   ##
============================================
- Coverage     90.00%   89.93%   -0.08%     
  Complexity     2660     2660              
============================================
  Files           310      310              
  Lines          8402     8402              
  Branches        838      838              
============================================
- Hits           7562     7556       -6     
- Misses          585      592       +7     
+ Partials        255      254       -1     
Impacted Files Coverage Δ Complexity Δ
...telemetry/sdk/trace/export/BatchSpanProcessor.java 84.21% <0.00%> (-4.39%) 10.00% <0.00%> (ø%)
...ntelemetry/exporter/zipkin/ZipkinSpanExporter.java 86.27% <0.00%> (-1.97%) 33.00% <0.00%> (-1.00%)
...dk/extension/trace/jaeger/sampler/RateLimiter.java 100.00% <0.00%> (+5.88%) 5.00% <0.00%> (+1.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 59bf511...7c06c65. Read the comment docs.

@bogdandrutu bogdandrutu merged commit 93dd38a into open-telemetry:main Feb 18, 2021
@bogdandrutu bogdandrutu deleted the paramsnotnull branch February 18, 2021 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants