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

[chore][exporter/opensearchexporter] use errors.Join instead of go.uber.org/multierr #27908

Merged
merged 3 commits into from
Oct 25, 2023

Conversation

bagmeg
Copy link
Contributor

@bagmeg bagmeg commented Oct 23, 2023

Description:
use errors.Join instead of go.uber.org/multierr

Link to tracking Issue:
#25121

Testing:

Documentation:

@bagmeg bagmeg requested a review from Aneurysm9 as a code owner October 23, 2023 11:35
@bagmeg bagmeg requested a review from a team October 23, 2023 11:35
@songy23 songy23 added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Oct 23, 2023
@bagmeg bagmeg changed the title use errors.Join instead of go.uber.org/multierr [chore][exporter/opensearchexporter] use errors.Join instead of go.uber.org/multierr Oct 23, 2023
Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @bagmeg, please resolve the conflict

@mx-psi mx-psi added the ready to merge Code review completed; ready to merge by maintainers label Oct 25, 2023
@mx-psi mx-psi merged commit 8a95aad into open-telemetry:main Oct 25, 2023
86 of 87 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 25, 2023
sigilioso pushed a commit to carlossscastro/opentelemetry-collector-contrib that referenced this pull request Oct 27, 2023
…er.org/multierr (open-telemetry#27908)

**Description:** <Describe what has changed.>
use errors.Join instead of go.uber.org/multierr

**Link to tracking Issue:** <Issue number if applicable>
open-telemetry#25121 

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>
jmsnll pushed a commit to jmsnll/opentelemetry-collector-contrib that referenced this pull request Nov 12, 2023
…er.org/multierr (open-telemetry#27908)

**Description:** <Describe what has changed.>
use errors.Join instead of go.uber.org/multierr

**Link to tracking Issue:** <Issue number if applicable>
open-telemetry#25121 

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>
RoryCrispin pushed a commit to ClickHouse/opentelemetry-collector-contrib that referenced this pull request Nov 24, 2023
…er.org/multierr (open-telemetry#27908)

**Description:** <Describe what has changed.>
use errors.Join instead of go.uber.org/multierr

**Link to tracking Issue:** <Issue number if applicable>
open-telemetry#25121 

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exporter/opensearch ready to merge Code review completed; ready to merge by maintainers Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants