Pablo Baeyens
Pablo Baeyens
Per https://github.com/open-telemetry/opentelemetry-collector/pull/11406#issuecomment-2493239777 I am merging this :tada:
We should have already addressed everything as part of #3839, but if there is anything missing we should do it now as well
@Sanket-0510 Not your fault at all! To be crystal clear: this is a problem with our CI and how we do testing and not with any individual contributor. Mistakes happen...
@Sanket-0510 Yes, in order to solve this issue this is one of the things that would help
@KenEkanem Assigned to you, a linter sounds like a good approach for this :)
CI should fail if an error is detected
@atoulme I want to work on improving the coding guidelines (specifically, making naming conventions clearer as we have discussed offline a couple times). This is the very first step, moving...
For resiliency, we may want to consider a requirement about persistent storage support (at least under graceful termination situations)
We may also want to have requirements about keeping the size of internal state in check for stateful components
I am going to keep this one open but remove this from Collector v1 since we have almost completed it and the last item refers to semconv which doesn't really...