-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Add jpkrohling as owner for some components #6531
Add jpkrohling as owner for some components #6531
Conversation
Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>
Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>
c849ad4
to
8d6783c
Compare
exporter/tanzuobservabilityexporter/ @open-telemetry/collector-contrib-approvers @oppegard @thepeterstone @keep94 | ||
exporter/tencentcloudlogserviceexporter/ @open-telemetry/collector-contrib-approvers @wgliang @yiyang5055 | ||
exporter/influxdbexporter/ @open-telemetry/collector-contrib-approvers @jacobmarble @8none1 | ||
exporter/uptraceexporter/ @open-telemetry/collector-contrib-approvers @vmihailenco |
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.
Looks like this was removed in #4458 so we can delete this line
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.
Approving, but left some comments that can either be dealt here or in a separate PR
Given the potential for conflicts, I'll merge this as is and send in a new PR with the requested changes. |
…6531) This change updates values returned by `StabilityLevel.String` to be consistent with other enum types: - All returned strings are capitalized. - "Undefined" is returned only for `StabilityLevelUndefined`. - "" is returned for integers that are out of StabilityLevel enum range.
Related to #3870
Signed-off-by: Juraci Paixão Kröhling juraci@kroehling.de