(refactor): Migrate all internal ITelemetryLoggerExt use to TelemetryLoggerExt#27484
Conversation
|
Hi! Thank you for opening this PR. Want me to review it? Based on the diff (587 lines, 108 files), I've queued these reviewers:
How this works
|
Fleet Review — CleanNo issues found across the reviewer fleet for this run. |
| import type { | ||
| ITelemetryLoggerExt, | ||
| TelemetryLoggerExt, | ||
| MonitoringContext, |
There was a problem hiding this comment.
imports are no longer sorted. Unfortunately, we don't have enforcement for that style. Most tooling I've seen will respect sort if everything already there is sorted. Otherwise, new stuff goes to the end.
Would be nice to restore sorting to changed blocks, but not high priority.
Description
This change is a follow up to #26912 that introduced the new internal
TelemetryLoggerExtinterface. Said interface can be used to replace calls to the internalITelemetryLoggerExtBreaking Change
This is a necessary change to enable this PR to be merged: #27476
Reviewer Guidance
The review process is outlined on this wiki page.
The change is just a naming one, no functionality was affected
Fixes: AB#74860