Adding telemetry for devcmd environment usage (#14023)

Author: bobbrow

Extension/src/LanguageServer/cppBuildTaskProvider.ts

@@ -436,6 +436,7 @@ class CustomBuildTaskTerminal implements Pseudoterminal {
             // However, this does not apply to processes that we spawn ourselves in the Pseudoterminal, so we need to specify the
             // correct environment in order to emulate the terminal behavior properly.
             this.options.env = getEffectiveEnvironment();
+            telemetry.logLanguageServerEvent('buildUsesEnvironmentOverride');
         }
 
         const splitWriteEmitter = (lines: string | Buffer) => {
@@ -498,18 +499,15 @@ class CustomBuildTaskTerminal implements Pseudoterminal {
 
     private printBuildSummary(error: string, stdout: string, stderr: string, spawnResult: number): number {
         if (spawnResult !== 0) {
-            telemetry.logLanguageServerEvent("cppBuildTaskError");
             this.writeEmitter.fire(localize("build.finished.with.error", "Build finished with error(s).") + this.endOfLine);
             return -1;
         }
         if (error || (!stdout && stderr && stderr.includes("error")) ||
             (stdout && (stdout.includes("error C") || stdout.includes("LINK : fatal error")))) { // cl.exe compiler errors
-            telemetry.logLanguageServerEvent("cppBuildTaskError");
             this.writeEmitter.fire(localize("build.finished.with.error", "Build finished with error(s).") + this.endOfLine);
             return -1;
         } else if ((!stdout && stderr) || // gcc/clang
             (stdout && stdout.includes("warning C"))) { // cl.exe compiler warnings
-            telemetry.logLanguageServerEvent("cppBuildTaskWarnings");
             this.writeEmitter.fire(localize("build.finished.with.warnings", "Build finished with warning(s).") + this.endOfLine);
             return 0;
         } else {