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

Rename *Provider names #1190

Merged
merged 3 commits into from
Sep 23, 2020
Merged

Rename *Provider names #1190

merged 3 commits into from
Sep 23, 2020

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Sep 21, 2020

There is overlap in naming with MeterProviders and TracerProviders. This is means the specification is not implemented and these types can not exist in the same package (#1179). This makes each type and related functions and types explicit.

Part of #1179 and #964

There is overlap in naming with MeterProviders and TracerProviders. This
is means the specification is not implemented and these types can not
exist in the same package (open-telemetry#1179). This makes each type and related
functions and types explicit.
@codecov
Copy link

codecov bot commented Sep 21, 2020

Codecov Report

Merging #1190 into master will increase coverage by 0.6%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #1190     +/-   ##
========================================
+ Coverage    76.3%   77.0%   +0.6%     
========================================
  Files         134     135      +1     
  Lines        5720    5867    +147     
========================================
+ Hits         4368    4521    +153     
+ Misses       1107    1104      -3     
+ Partials      245     242      -3     
Impacted Files Coverage Δ
api/global/internal/meter.go 90.3% <ø> (ø)
api/global/internal/state.go 84.2% <ø> (ø)
api/global/internal/trace.go 88.8% <ø> (ø)
api/global/metric.go 100.0% <ø> (ø)
api/global/trace.go 66.6% <ø> (ø)
api/metric/meter.go 57.7% <ø> (ø)
api/metric/metrictest/meter.go 100.0% <ø> (ø)
api/metric/noop.go 0.0% <ø> (ø)
api/metric/registry/registry.go 92.1% <ø> (ø)
api/trace/api.go 57.1% <ø> (ø)
... and 20 more

@MrAlias MrAlias added area:metrics Part of OpenTelemetry Metrics area:trace Part of OpenTelemetry tracing pkg:API Related to an API package pkg:SDK Related to an SDK package priority:p1 labels Sep 21, 2020
@MrAlias MrAlias self-assigned this Sep 21, 2020
@MrAlias MrAlias added this to the RC1 milestone Sep 21, 2020
@MrAlias MrAlias marked this pull request as ready for review September 21, 2020 17:42
@Aneurysm9 Aneurysm9 merged commit 7d71867 into open-telemetry:master Sep 23, 2020
@MrAlias MrAlias deleted the providers branch September 24, 2020 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:metrics Part of OpenTelemetry Metrics area:trace Part of OpenTelemetry tracing pkg:API Related to an API package pkg:SDK Related to an SDK package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants