Arthur Silva Sens

Results 418 comments of Arthur Silva Sens

@krajorama, do you want to take another look or am I ok to merge?

> Can you please explain why #39753 would depend on constants being moved to `otlptranslator`? I fail to see this being the case. It's not a requirement, it's an idea...

Argh, looks like we need to update otlptranslator in Prometheus first, and then we can update it here 😬

urgh, `make crosslink` is panicking for me 😕

> I also don't get #1641 goal. I thought this issue is to fail on registration, which is generally best-effort (e.g. there are cases of unchecked collectors etc) and that's...

Sure thing! Is the discussion here enough for you to start working, or do you feel like you need extra guidance?

Yeah, it's on my list for quite a while now and I don't even remember what blocked me 😓 I'll try to get back to this at some point this...

Alright, finally found time to come back to this :) I had to modify the approach a bit after #1476. I'm now generating the `withDebugMetrics()` function with the script that...

If everything looks good, one strategy is to merge this PR instead of the other two. This way we get all commits in without needing to rebase several PRs.