Skip to content

Drop Microsoft.ApplicationInsights — telemetry is a no-op #79

Description

@ChrisonSimtian

Telemetry.cs short-circuits because InstrumentationKey = "" (per the rebrand note: original NUKE key was matkoch-owned, we don't reuse). The Microsoft.ApplicationInsights PackageVersion is dead weight until we stand up a Fallout-owned endpoint.

Drop the package + the unused fields/methods in Telemetry.cs that reference TelemetryClient. When a Fallout telemetry backend lands, re-introduce.

Refs the dependency audit. Done when:

  • PackageVersion removed from Directory.Packages.props
  • PackageReference (if direct) removed
  • TelemetryClient s_client and call sites either removed or stubbed
  • Build clean, tests pass

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions