Skip to content

Move Version function and code from global to toplevel#1330

Merged
MrAlias merged 4 commits intoopen-telemetry:masterfrom
krnowak:new-api-5
Nov 16, 2020
Merged

Move Version function and code from global to toplevel#1330
MrAlias merged 4 commits intoopen-telemetry:masterfrom
krnowak:new-api-5

Conversation

@krnowak
Copy link
Copy Markdown
Member

@krnowak krnowak commented Nov 13, 2020

This is a last code-moving PR that is a part of #1303. Next PR coming after this one will be about package docs update.

Resolves #1286
Resolves #1009

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 13, 2020

Codecov Report

Merging #1330 (1d59ff1) into master (20c78fe) will decrease coverage by 0.0%.
The diff coverage is 46.5%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #1330     +/-   ##
========================================
- Coverage    77.3%   77.3%   -0.1%     
========================================
  Files         123     123             
  Lines        6067    6067             
========================================
- Hits         4694    4693      -1     
- Misses       1120    1121      +1     
  Partials      253     253             
Impacted Files Coverage Δ
bridge/opencensus/bridge.go 80.8% <0.0%> (ø)
bridge/opentracing/bridge.go 39.0% <0.0%> (ø)
exporters/metric/prometheus/prometheus.go 57.6% <0.0%> (ø)
exporters/stdout/exporter.go 25.9% <0.0%> (ø)
internal/global/meter.go 90.3% <ø> (ø)
internal/global/propagator.go 100.0% <ø> (ø)
internal/global/state.go 100.0% <ø> (ø)
internal/global/trace.go 88.8% <ø> (ø)
internal/metric/async.go 77.7% <0.0%> (ø)
propagation.go 0.0% <0.0%> (ø)
... and 16 more

Comment thread doc.go
@MrAlias MrAlias added pkg:API Related to an API package priority:p1 labels Nov 13, 2020
@MrAlias MrAlias added this to the RC1 milestone Nov 13, 2020
@krnowak
Copy link
Copy Markdown
Member Author

krnowak commented Nov 16, 2020

Did a push to kick the hanged easycla task.

@MrAlias MrAlias merged commit 3d6e8b5 into open-telemetry:master Nov 16, 2020
@krnowak krnowak deleted the new-api-5 branch November 16, 2020 20:23
@MrAlias MrAlias mentioned this pull request Nov 20, 2020
AzfaarQureshi pushed a commit to open-o11y/opentelemetry-go that referenced this pull request Dec 3, 2020
…y#1330)

* Move global code to toplevel package

* Move version function to toplevel package

* Update changelog

Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg:API Related to an API package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The preferred error handler RFC to rename global package

3 participants