Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pass res to exporter #3250

merged 1 commit into from Oct 2, 2022

pass res to exporter #3250

merged 1 commit into from Oct 2, 2022


Copy link

@ktong ktong commented Sep 30, 2022

Follow #3206 to pass res to &exporter

Copy link

linux-foundation-easycla bot commented Sep 30, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: ktong / name: Kuisong Tong (4d95412)

Copy link

codecov bot commented Sep 30, 2022

Codecov Report

Merging #3250 (4d95412) into main (aca054b) will not change coverage.
The diff coverage is 0.0%.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #3250   +/-   ##
  Coverage   77.3%   77.3%           
  Files        159     159           
  Lines      11184   11184           
  Hits        8651    8651           
  Misses      2335    2335           
  Partials     198     198           
Impacted Files Coverage Δ
bridge/opencensus/metric.go 0.0% <0.0%> (ø)

Copy link

@dashpole dashpole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I had included this in #3207, but it is good to fix here.

@MrAlias MrAlias added the Skip Changelog PRs that do not require a entry label Oct 2, 2022
@MrAlias MrAlias merged commit efdbe5a into open-telemetry:main Oct 2, 2022
@ktong ktong deleted the patch-1 branch October 2, 2022 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Skip Changelog PRs that do not require a entry
None yet

Successfully merging this pull request may close these issues.

None yet

3 participants