Skip to content
This repository has been archived by the owner on Dec 23, 2023. It is now read-only.

Add a warning about try-with-resource to auto-close spans. #1362

Merged
merged 1 commit into from
Aug 6, 2018

Conversation

bogdandrutu
Copy link
Contributor

Fixes #646

Thanks @adriancole for reporting this a while ago.

@codecov-io
Copy link

codecov-io commented Aug 3, 2018

Codecov Report

Merging #1362 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             master   #1362   +/-   ##
========================================
  Coverage      83.2%   83.2%           
  Complexity     1393    1393           
========================================
  Files           221     221           
  Lines          6586    6586           
  Branches        620     620           
========================================
  Hits           5480    5480           
  Misses          930     930           
  Partials        176     176
Impacted Files Coverage Δ Complexity Δ
...src/main/java/io/opencensus/trace/SpanBuilder.java 94.44% <ø> (ø) 4 <0> (ø) ⬇️

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 b0c487b...171f650. Read the comment docs.

@bogdandrutu bogdandrutu merged commit c943c6c into census-instrumentation:master Aug 6, 2018
@bogdandrutu bogdandrutu deleted the trywith branch August 6, 2018 23:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants