-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[connector,exporter,receiver,extension,processor] remove deprecated funcs/structs #10423
[connector,exporter,receiver,extension,processor] remove deprecated funcs/structs #10423
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10423 +/- ##
==========================================
+ Coverage 92.33% 92.38% +0.04%
==========================================
Files 403 403
Lines 18734 18724 -10
==========================================
Hits 17298 17298
+ Misses 1076 1066 -10
Partials 360 360 ☔ View full report in Codecov by Sentry. |
This dependency broke because of open-telemetry/opentelemetry-collector#10423, bumping the dep to unblock the core PR. Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
This dependency broke because of open-telemetry/opentelemetry-collector#10423, bumping the dep to unblock the core PR. Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Blocked by #10447 |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
I think this is no longer blocked, right? |
…uncs/structs Finishing CreateSettings rename. Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
7f53d37
to
17c721f
Compare
|
@mx-psi pulled in the PR that removed the dependency. will see if it passes the CI now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It did! 🚀
Finishing CreateSettings rename.