Skip to content

feat: finalize logger#213

Closed
yuzurihaaa wants to merge 71 commits into
Workiva:masterfrom
yuzurihaaa:feat/finalize-logger
Closed

feat: finalize logger#213
yuzurihaaa wants to merge 71 commits into
Workiva:masterfrom
yuzurihaaa:feat/finalize-logger

Conversation

@yuzurihaaa
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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

@jodinathan
Copy link
Copy Markdown

is the branch working or is there anything else to do?

would be very nice to have this in the package!

@yuzurihaaa
Copy link
Copy Markdown
Contributor Author

yuzurihaaa commented Mar 3, 2025

@jodinathan afaik, the branch is working, but it is behind changes after review from the other branches. Do try it out and if u happens to stumble into any issue, let me know.

update: I've rebased with the changes from previous branches, but have yet to try it out into any logging services.

@yuzurihaaa yuzurihaaa force-pushed the feat/finalize-logger branch from 940030c to efe0871 Compare March 3, 2025 02:57
@yuzurihaaa yuzurihaaa force-pushed the feat/finalize-logger branch 2 times, most recently from de01f9e to fc34a54 Compare March 18, 2025 05:05
@yuzurihaaa yuzurihaaa force-pushed the feat/finalize-logger branch from fc34a54 to a985244 Compare March 18, 2025 05:08
@dustinlessard-wf
Copy link
Copy Markdown
Contributor

Hi @yuzurihaaa
Thank you for taking the time to contribute to this project. We really appreciate the effort you put into opening this pull request.
I want to be transparent about the state of the repository: we have not been actively maintaining this codebase, and we do not currently have the bandwidth to properly review, test, and support external contributions at the standard they deserve.
For that reason, we’re going to close this pull request. This is not a reflection on the quality or value of your contribution. Rather, it reflects our current inability to responsibly vet and merge changes into the project.
We are also exploring a path to transfer ownership or stewardship of the project outside of our company, so that it can hopefully have a more active and sustainable future. Until that direction is clearer, we do not want to create a misleading impression that the repository is actively maintained or that we can support merged changes going forward.
Thank you again for your contribution and for your interest in improving the project. We’re sorry we were not able to engage with your pull request in the way it deserved.

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

5 participants