krishan kumar
krishan kumar
Yeah, we can wrap the existing package. It gives us what we need for now, and we can add our own features on top.
https://dev.to/kkumargcc/build-a-command-line-todo-app-with-goravel-console-3l61
> The report is sent by default currently, it should be unexpected. Could you optimize it in this PR? > > > > > > Is it for grpc?
Okay, I’ll check. I’m thinking of using a kill switch for all instrumentation in `config/telemetry.go` to disable telemetry globally. This switch would override all other configuration and help enable or...
> The report is sent by default currently, it should be unexpected. Could you optimize it in this PR? > @hwbrzzl I think the issue is in `telemetry.metrics.exporter`. We are...
@coderabbitai review
I recommend against merging methods that are ambiguous, handle edge cases incorrectly, or introduce significant performance bottlenecks. It is better to omit these features than to expose unsafe or unoptimized...