Remove superfluous and flawed telemetry event (#821)

Thanks @minestarks for pointing this out. This event only fires when
cells are set to q# for the first
time, not on subsequent cell accesses. This isn't particularly useful
telemetry. Additionally, this event
is superseded by [this
event](https://github.com/microsoft/qsharp/blob/main/vscode/src/extension.ts#L121).

vscode/src/notebook.ts

@@ -3,7 +3,6 @@
 
 import * as vscode from "vscode";
 import { qsharpLanguageId } from "./common.js";
-import { EventType, sendTelemetryEvent } from "./telemetry.js";
 import { WorkspaceTreeProvider } from "./azure/treeView.js";
 import { getPythonCodeForWorkspace } from "./azure/workspaceActions.js";
 import { notebookTemplate } from "./notebookTemplate.js";
@@ -58,7 +57,6 @@ export function registerQSharpNotebookHandlers() {
             cell.document,
             qsharpLanguageId,
           );
-          sendTelemetryEvent(EventType.QSharpJupyterCellInitialized);
         }
       }
     }

vscode/src/telemetry.ts

@@ -5,7 +5,6 @@ import { log } from "qsharp-lang";
 export enum EventType {
   InitializePlugin = "Qsharp.InitializePlugin",
   LoadLanguageService = "Qsharp.LoadLanguageService",
-  QSharpJupyterCellInitialized = "Qsharp.JupyterCellInitialized",
   ReturnCompletionList = "Qsharp.ReturnCompletionList",
   GenerateQirStart = "Qsharp.GenerateQirStart",
   GenerateQirEnd = "Qsharp.GenerateQirEnd",
@@ -45,10 +44,6 @@ type EventTypes = {
       timeToStartMs: number;
     };
   };
-  [EventType.QSharpJupyterCellInitialized]: {
-    properties: Empty;
-    measurements: Empty;
-  };
   [EventType.ReturnCompletionList]: {
     properties: Empty;
     measurements: { timeToCompletionMs: number; completionListLength: number };