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

feat: finalize logger #213

Open
wants to merge 32 commits into
base: master
Choose a base branch
from

Conversation

yuzurihaaa
Copy link
Contributor

Which problem is this PR solving?

  1. Continuation of feat: otlp logger exporter #212
  2. Mark Log as Beta

Fixes # (issue)
close #172

Short description of the change

  1. Change from experimental_api/sdk to api/sdk.
  2. Update README

How Has This Been Tested?

N/A

Checklist:

  • Unit tests have been added
  • Documentation has been updated

@tigloo
Copy link

tigloo commented Jan 30, 2025

Is any help needed to finish the logging PRs? We would really like to see the logging feature included in the package, as it would allow us to move completely over to Open Telemetry.

@yuzurihaaa
Copy link
Contributor Author

Hi @tigloo , all the PRs here https://github.com/Workiva/opentelemetry-dart/pulls/yuzurihaaa just with help on fixing the reviews or even help by reviewing the PRs, that would be helpful to get this completed fast 😄

I am trying my best to get this completed too because I am doing some extra package for offline first logs https://github.com/feedmepos/offline-otlp-log-exporter

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.

Log feature
3 participants