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

[WIP] Move MeterProvider to API #592

Closed
wants to merge 1 commit into from
Closed

[WIP] Move MeterProvider to API #592

wants to merge 1 commit into from

Conversation

eddynaka
Copy link
Contributor

Fix #579

@MikeGoldsmith
Copy link
Member

Thanks for the start @eddynaka.

I agree with @cijothomas's comments in the issue and maybe the issue wasn't overly clear. The purpose of the issue was to move the API specific components to the API level, things that are the building blocks so if someone wanted to create an alternative SDK, they could do so.

I think we should put a pin in this one until @cijothomas's coming PRs are complete and we can review what should be in the API.

@eddynaka
Copy link
Contributor Author

@MikeGoldsmith , yeah, i talked with @cijothomas about this. i will probably close and wait for his PR.

Thanks again! :)

@eddynaka eddynaka closed this Apr 14, 2020
@eddynaka eddynaka deleted the feature/move-meter-provider-to-api branch April 14, 2020 11:11
Yun-Ting pushed a commit to Yun-Ting/opentelemetry-dotnet that referenced this pull request Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move MeterProvider to API
2 participants