cmd/templ/lspcmd/proxy: set 'templ' as client name in initialize client info#966
cmd/templ/lspcmd/proxy: set 'templ' as client name in initialize client info#966
Conversation
|
Ah, good idea! I'm not familiar with the format of adding a Do you any links to the telemetry dataset etc. or conversations about that? The reason I ask is that once it's set, it's probably hard to change, since it would affect historic data collection, so I just want to cover that before merging. I'd like to see telemetry of the templ LSP. For example, it may be possible to see new issues early using the telemetry. |
This allows gopls to recognize gopls instances serving for templ and count separately in go telemetry data processing.
per discussion in github.com/golang/go/issues/70205
|
Hi @a-h |
|
Thanks! I'll probably be able to look at this the day after tomorrow. |
|
@hyangah what's the status on this now then? I think we're still waiting on the discussion in golang/go#70205 to resolve. I've added a comment there to explain how templ works etc. |
This allows gopls to recognize gopls instances serving for templ and count separately in go telemetry data processing.