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.

Sign up for GitHub

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

Jump to bottom

Deprecate and remove processorhelper.NewUnhealthyProcessorCreateSettings #11307

Closed
bogdandrutu opened this issue Sep 30, 2024 · 4 comments · Fixed by #11376
Closed

Deprecate and remove processorhelper.NewUnhealthyProcessorCreateSettings #11307

bogdandrutu opened this issue Sep 30, 2024 · 4 comments · Fixed by #11376
Assignees
@zzzk1
Labels
good first issue Good for newcomers help wanted Good issue for contributors to OpenTelemetry Service to pick up

Comments

@bogdandrutu
Copy link
Member

Not used and equivalent with NewNopSettings
@bogdandrutu bogdandrutu added help wanted Good issue for contributors to OpenTelemetry Service to pick up good first issue Good for newcomers labels Sep 30, 2024
@zzzk1
Copy link
Contributor

zzzk1 commented Oct 1, 2024

@bogdandrutu, may I give it a try?

@olamideTiana
Copy link
Contributor

@bogdandrutu Can I work on this as well?

@bogdandrutu
Copy link
Member Author

@olamideTiana you can give this a try, I don't see yet a PR from @zzzk1

@olamideTiana
Copy link
Contributor

Thank you @bogdandrutu

@zzzk1 zzzk1 mentioned this issue Oct 7, 2024
Closed
@codeboten codeboten mentioned this issue Oct 7, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zzzk1 zzzk1

Labels
good first issue Good for newcomers help wanted Good issue for contributors to OpenTelemetry Service to pick up
Projects
None yet
Milestone
No milestone
3 participants
@bogdandrutu @zzzk1 @olamideTiana