Pablo Baeyens
Pablo Baeyens
Go 1.20 includes new builtin [error joining support](https://pkg.go.dev/errors#Join) that can replace the usage of `go.uber.org/multierr`. We should switch to using these and ban the usage of `go.uber.org/multierr` via `depguard` (same...
We currently don't have a way to skip contrib tests in the merge queue. We usually don't need this, but whenever we do, we need to add a way to...
Clarifies what sections from the project template are required, and which are to be filled after approval
It would be great to have a triaging document of some kind describing the areas that issues can fall into and what the process is. I am going to start...
## Description Adds `configoptional`. Tests it on `confighttp` and `otlpreceiver` modules. Thanks @jade-guiton-dd for the feedback and help with designing this, and thanks to @yurishkuro for the initial version (see...
#### Description Adds a `confmap.Marshaler` implementation for `configoptional.Optional`
#### Description Makes the following changes to confighttp: 1. Uses `configoptional.Optional` in `confighttp` for all optional sections This means a bunch of breaking changes. I think it is unfeasible to...
Add testable examples to https://pkg.go.dev/go.opentelemetry.io/collector/pdata. There's a lot of things that would benefit from examples here, I suggest to start with a single type and add examples for it
Not something we have to decide here, and it's minor since it's largely internal, but how would you feel about calling this variable `hasValue` and renaming `hasValue` to `hasSome`? 1....
We could simplify these cases by extending the `Get()` to return (T, bool) instead. _Originally posted by @rogercoll in https://github.com/open-telemetry/opentelemetry-collector/pull/13119#discussion_r2126336996_