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

Permit keyword list start_opts in Tracer macros. #228

Merged
merged 2 commits into from
Mar 24, 2021

Conversation

garthk
Copy link
Contributor

@garthk garthk commented Mar 22, 2021

Ergonomic for Elixir developers.

Also:

  • Repair test name to reflect test code

  • Provide tested examples of with_span/3 with start_opts expressed as both a map and keyword list

@codecov
Copy link

codecov bot commented Mar 22, 2021

Codecov Report

Merging #228 (32e85ef) into main (4889eda) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #228   +/-   ##
=======================================
  Coverage   34.73%   34.73%           
=======================================
  Files          37       37           
  Lines        3052     3052           
=======================================
  Hits         1060     1060           
  Misses       1992     1992           
Flag Coverage Δ
api 61.11% <ø> (ø)
elixir 18.33% <ø> (+0.27%) ⬆️
erlang 34.58% <ø> (ø)
exporter 19.48% <ø> (ø)
sdk 77.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...pps/opentelemetry_api/lib/open_telemetry/tracer.ex 46.15% <ø> (ø)

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 4889eda...32e85ef. Read the comment docs.

@garthk garthk force-pushed the feat/permit-keyword-list-start-opts branch from 6596930 to ae9062c Compare March 22, 2021 22:50
Ergonomic for Elixir developers.

Also:

* Repair test name to reflect test code

* Provide tested examples of `with_span/3` with `start_opts` expressed
  as both a map and keyword list
@garthk garthk force-pushed the feat/permit-keyword-list-start-opts branch from ae9062c to f9ab9b9 Compare March 23, 2021 22:40
@ferd ferd merged commit ff182a7 into open-telemetry:main Mar 24, 2021
@garthk garthk deleted the feat/permit-keyword-list-start-opts branch March 24, 2021 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants