Skip to content

Permit keyword list start_opts in Tracer macros. - #228

Merged
ferd merged 2 commits into
open-telemetry:mainfrom
garthk:feat/permit-keyword-list-start-opts
Mar 24, 2021
Merged

Permit keyword list start_opts in Tracer macros.#228
ferd merged 2 commits into
open-telemetry:mainfrom
garthk:feat/permit-keyword-list-start-opts

Conversation

@garthk

@garthk garthk commented Mar 22, 2021

Copy link
Copy Markdown
Contributor

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

codecov Bot commented Mar 22, 2021

Copy link
Copy Markdown

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