Removed AppInsights

madskristensen · madskristensen · commit 11d36280f106 · 2016-08-02T10:31:47.000-07:00
diff --git a/src/WebCompiler/Config/ConfigFileProcessor.cs b/src/WebCompiler/Config/ConfigFileProcessor.cs
@@ -123,7 +123,7 @@ private IEnumerable<CompilerResult> SourceFileChanged(string configFile,
 
                 //compile files that are dependent on the current file
                 var dependencies = DependencyService.GetDependencies(projectPath, sourceFile);
-                if(dependencies != null)
+                if (dependencies != null)
                 {
                     string key = sourceFile.ToLowerInvariant();
 
@@ -179,9 +179,8 @@ public static IEnumerable<Config> IsFileConfigured(string configFile, string sou
 
                 return list;
             }
-            catch (Exception ex)
+            catch (Exception)
             {
-                Telemetry.TrackException(ex);
                 return null;
             }
         }
@@ -240,8 +239,6 @@ private CompilerResult ProcessConfig(string baseFolder, Config config)
                 OnAfterWritingSourceMap(absolute, mapFile, smChanges);
             }
 
-            Telemetry.TrackCompile(config);
-
             return result;
         }
 
diff --git a/src/WebCompiler/MSBuild/CompilerBuildTask.cs b/src/WebCompiler/MSBuild/CompilerBuildTask.cs
@@ -29,7 +29,6 @@ public override bool Execute()
             }
 
             Log.LogMessage(MessageImportance.High, Environment.NewLine + "WebCompiler: Begin compiling " + configFile.Name);
-            Telemetry.SetDeviceName("MSBuild");
 
             ConfigFileProcessor processor = new ConfigFileProcessor();
             processor.BeforeProcess += (s, e) => { if (e.ContainsChanges) FileHelpers.RemoveReadonlyFlagFromFile(e.Config.GetAbsoluteOutputFile()); };
@@ -68,7 +67,6 @@ public override bool Execute()
             catch (Exception ex)
             {
                 Log.LogError(ex.Message);
-                Telemetry.TrackException(ex);
                 return false;
             }
         }
diff --git a/src/WebCompiler/MSBuild/CompilerCleanTask.cs b/src/WebCompiler/MSBuild/CompilerCleanTask.cs
@@ -29,7 +29,6 @@ public override bool Execute()
             }
 
             Log.LogMessage(MessageImportance.High, Environment.NewLine + "WebCompiler: Begin cleaning output of " + configFile.Name);
-            Telemetry.SetDeviceName("MSBuild");
 
             try
             {
@@ -43,7 +42,6 @@ public override bool Execute()
             catch (Exception ex)
             {
                 Log.LogError(ex.Message);
-                Telemetry.TrackException(ex);
                 return false;
             }
         }
diff --git a/src/WebCompiler/Program.cs b/src/WebCompiler/Program.cs
@@ -19,8 +19,6 @@ static int Main(params string[] args)
                 return 0;
             }
 
-            Telemetry.SetDeviceName("Command line");
-
             ConfigFileProcessor processor = new ConfigFileProcessor();
             EventHookups(processor, configPath);
 
diff --git a/src/WebCompiler/Telemetry.cs b/src/WebCompiler/Telemetry.cs
diff --git a/src/WebCompiler/WebCompiler.csproj b/src/WebCompiler/WebCompiler.csproj
@@ -42,10 +42,6 @@
       <HintPath>..\..\packages\AjaxMin.5.14.5506.26202\lib\net40\AjaxMin.dll</HintPath>
       <Private>True</Private>
     </Reference>
-    <Reference Include="Microsoft.ApplicationInsights, Version=1.2.0.5639, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL">
-      <HintPath>..\..\packages\Microsoft.ApplicationInsights.1.2.0\lib\net45\Microsoft.ApplicationInsights.dll</HintPath>
-      <Private>True</Private>
-    </Reference>
     <Reference Include="Microsoft.Build.Framework" />
     <Reference Include="Microsoft.Build.Tasks.v4.0" />
     <Reference Include="Microsoft.Build.Utilities.v4.0" />
@@ -102,7 +98,6 @@
     <Compile Include="Config\ConfigHandler.cs" />
     <Compile Include="Config\ConfigFileProcessor.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
-    <Compile Include="Telemetry.cs" />
   </ItemGroup>
   <ItemGroup>
     <None Include="MSBuild\BuildWebCompiler.nuspec">
diff --git a/src/WebCompiler/packages.config b/src/WebCompiler/packages.config
@@ -1,7 +1,6 @@
 ﻿<?xml version="1.0" encoding="utf-8"?>
 <packages>
   <package id="AjaxMin" version="5.14.5506.26202" targetFramework="net45" />
-  <package id="Microsoft.ApplicationInsights" version="1.2.0" targetFramework="net45" />
   <package id="Newtonsoft.Json" version="7.0.1" targetFramework="net45" />
   <package id="NuGet.CommandLine" version="2.8.5" targetFramework="net45" />
 </packages>
diff --git a/src/WebCompilerTest/AssemblyLoad.cs b/src/WebCompilerTest/AssemblyLoad.cs
diff --git a/src/WebCompilerTest/Compile/BabelTest.cs b/src/WebCompilerTest/Compile/BabelTest.cs
@@ -13,7 +13,6 @@ public class BabelTest
         [TestInitialize]
         public void Setup()
         {
-            Telemetry.Enabled = false;
             _processor = new ConfigFileProcessor();
         }
 
diff --git a/src/WebCompilerTest/Compile/CoffeeScriptTest.cs b/src/WebCompilerTest/Compile/CoffeeScriptTest.cs
@@ -13,7 +13,6 @@ public class CoffeeScriptTest
         [TestInitialize]
         public void Setup()
         {
-            Telemetry.Enabled = false;
             _processor = new ConfigFileProcessor();
         }
 
diff --git a/src/WebCompilerTest/Compile/CompileServiceTest.cs b/src/WebCompilerTest/Compile/CompileServiceTest.cs
@@ -6,12 +6,6 @@ namespace WebCompilerTest
     [TestClass]
     public class CompileServiceTest
     {
-        [TestInitialize]
-        public void Setup()
-        {
-            Telemetry.Enabled = false;
-        }
-
         [TestMethod, TestCategory("CompileService")]
         public void LessIsSupported()
         {
diff --git a/src/WebCompilerTest/Compile/IcedCoffeeScriptTest.cs b/src/WebCompilerTest/Compile/IcedCoffeeScriptTest.cs
@@ -13,7 +13,6 @@ public class IcedCoffeeScriptTest
         [TestInitialize]
         public void Setup()
         {
-            Telemetry.Enabled = false;
             _processor = new ConfigFileProcessor();
         }
 
diff --git a/src/WebCompilerTest/Compile/LessOptionsTest.cs b/src/WebCompilerTest/Compile/LessOptionsTest.cs
@@ -7,12 +7,6 @@ namespace WebCompilerTest
     [TestClass]
     public class LessOptionsTest
     {
-        [TestInitialize]
-        public void Setup()
-        {
-            Telemetry.Enabled = false;
-        }
-
         [TestMethod, TestCategory("LessOptions")]
         public void RelativeUrls()
         {
diff --git a/src/WebCompilerTest/Compile/LessTest.cs b/src/WebCompilerTest/Compile/LessTest.cs
@@ -13,7 +13,6 @@ public class LessTest
         [TestInitialize]
         public void Setup()
         {
-            Telemetry.Enabled = false;
             _processor = new ConfigFileProcessor();
             Cleanup();
         }
diff --git a/src/WebCompilerTest/Compile/ScssOptionsTest.cs b/src/WebCompilerTest/Compile/ScssOptionsTest.cs
@@ -7,12 +7,6 @@ namespace WebCompilerTest
     [TestClass]
     public class ScssOptionsTest
     {
-        [TestInitialize]
-        public void Setup()
-        {
-            Telemetry.Enabled = false;
-        }
-
         [TestMethod, TestCategory("SCSSOptions")]
         public void OutputStyleNested()
         {
diff --git a/src/WebCompilerTest/Compile/ScssTest.cs b/src/WebCompilerTest/Compile/ScssTest.cs
@@ -15,7 +15,6 @@ public class ScssTest
         [TestInitialize]
         public void Setup()
         {
-            Telemetry.Enabled = false;
             _processor = new ConfigFileProcessor();
         }
 
diff --git a/src/WebCompilerTest/Compile/StylusTest.cs b/src/WebCompilerTest/Compile/StylusTest.cs
@@ -13,7 +13,6 @@ public class StylusTest
         [TestInitialize]
         public void Setup()
         {
-            Telemetry.Enabled = false;
             _processor = new ConfigFileProcessor();
         }
 
diff --git a/src/WebCompilerTest/Config/ConfigHandlerTest.cs b/src/WebCompilerTest/Config/ConfigHandlerTest.cs
@@ -16,7 +16,6 @@ public class ConfigHandlerTest
         [TestInitialize]
         public void Setup()
         {
-            Telemetry.Enabled = false;
             _handler = new ConfigHandler();
 
             File.Copy(originalConfigFile, processingConfigFile, true);
diff --git a/src/WebCompilerTest/Minify/CssOptionsTests.cs b/src/WebCompilerTest/Minify/CssOptionsTests.cs
@@ -11,13 +11,6 @@ public class CssOptionsTests
     {
         private const string processingConfigFile = "../../Minify/artifacts/css/";
 
-        [TestInitialize]
-        public void Setup()
-        {
-            Telemetry.Enabled = false;
-        }
-
-
         [TestMethod, TestCategory("CssOptions")]
         public void CssCommentInUpperCaseShouldWork()
         {
diff --git a/src/WebCompilerTest/Minify/JavaScriptOptionsTests.cs b/src/WebCompilerTest/Minify/JavaScriptOptionsTests.cs
@@ -11,12 +11,6 @@ public class JavaScriptOptionsTests
     {
         private const string processingConfigFile = "../../Minify/artifacts/javascript/";
 
-        [TestInitialize]
-        public void Setup()
-        {
-            Telemetry.Enabled = false;
-        }
-
         [TestMethod, TestCategory("JavaScriptOptions")]
         public void EvanTreatmentInUpperCaseShouldWork()
         {
diff --git a/src/WebCompilerTest/WebCompilerTest.csproj b/src/WebCompilerTest/WebCompilerTest.csproj
@@ -55,7 +55,6 @@
     </Otherwise>
   </Choose>
   <ItemGroup>
-    <Compile Include="AssemblyLoad.cs" />
     <Compile Include="Compile\StylusTest.cs" />
     <Compile Include="Compile\IcedCoffeeScriptTest.cs" />
     <Compile Include="Compile\CoffeeScriptTest.cs" />
diff --git a/src/WebCompilerVsix/Commands/CleanOutputFiles.cs b/src/WebCompilerVsix/Commands/CleanOutputFiles.cs
@@ -87,8 +87,6 @@ private void AddConfig(object sender, EventArgs e)
                 string outputFile = config.GetAbsoluteOutputFile().FullName;
                 ProjectHelpers.DeleteFileFromProject(outputFile);
             }
-
-            Telemetry.TrackEvent("VS clean output files");
         }
     }
 }
diff --git a/src/WebCompilerVsix/Commands/CompileAllFiles.cs b/src/WebCompilerVsix/Commands/CompileAllFiles.cs
@@ -66,8 +66,6 @@ private void CompileSolution(object sender, EventArgs e)
                         CompilerService.Process(config);
                 }
             }
-
-            Telemetry.TrackEvent("VS compile solution");
         }
 
         private static List<string> GetFiles(string path, string pattern)
diff --git a/src/WebCompilerVsix/Commands/CompileOnBuild.cs b/src/WebCompilerVsix/Commands/CompileOnBuild.cs
@@ -135,7 +135,6 @@ private void EnableCompileOnBuild(object sender, EventArgs e)
                 if (question == DialogResult.No)
                     return;
 
-                Telemetry.TrackEvent("VS add compile on build");
                 Version version = new Version(WebCompilerPackage.Version);
 
 #if DEBUG
@@ -166,8 +165,6 @@ private void EnableCompileOnBuild(object sender, EventArgs e)
             }
             else
             {
-                Telemetry.TrackEvent("VS remove compile on build");
-
                 System.Threading.ThreadPool.QueueUserWorkItem((o) =>
                 {
                     try
diff --git a/src/WebCompilerVsix/Commands/CreateConfig.cs b/src/WebCompilerVsix/Commands/CreateConfig.cs
@@ -117,7 +117,6 @@ private void AddConfig(object sender, EventArgs e)
             if (_reCompileConfigs.Any())
             {
                 string absoluteFile = Path.Combine(folder, relativeFile).Replace("/", "\\");
-                Telemetry.TrackEvent("VS recompile config");
                 CompilerService.SourceFileChanged(configFile, absoluteFile);
                 return;
             }
@@ -151,8 +150,6 @@ private void AddConfig(object sender, EventArgs e)
 
             CompilerService.Process(configFile, new[] { config });
             WebCompilerPackage._dte.StatusBar.Progress(false, "Compiling file");
-
-            Telemetry.TrackEvent("VS config created");
         }
 
         private static Config CreateConfigFile(string inputfile, string outputFile)
diff --git a/src/WebCompilerVsix/Commands/Recompile.cs b/src/WebCompilerVsix/Commands/Recompile.cs
@@ -87,8 +87,6 @@ private void UpdateSelectedConfig(object sender, EventArgs e)
                     file = project.GetConfigFile();
             }
 
-            Telemetry.TrackEvent("VS re-compile");
-
             if (!string.IsNullOrEmpty(file))
                 CompilerService.Process(file, force: true);
         }
diff --git a/src/WebCompilerVsix/Commands/RemoveConfig.cs b/src/WebCompilerVsix/Commands/RemoveConfig.cs
@@ -88,7 +88,6 @@ private void AddConfig(object sender, EventArgs e)
                 return;
 
             ConfigHandler handler = new ConfigHandler();
-            Telemetry.TrackEvent("VS remove config");
 
             try
             {
diff --git a/src/WebCompilerVsix/Logger.cs b/src/WebCompilerVsix/Logger.cs
diff --git a/src/WebCompilerVsix/WebCompilerPackage.cs b/src/WebCompilerVsix/WebCompilerPackage.cs

Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ private IEnumerable<CompilerResult> SourceFileChanged(string configFile,`
`123`	`123`
`124`	`124`	`//compile files that are dependent on the current file`
`125`	`125`	`var dependencies = DependencyService.GetDependencies(projectPath, sourceFile);`
`126`		`- if(dependencies != null)`
	`126`	`+ if (dependencies != null)`
`127`	`127`	`{`
`128`	`128`	`string key = sourceFile.ToLowerInvariant();`
`129`	`129`
`@@ -179,9 +179,8 @@ public static IEnumerable<Config> IsFileConfigured(string configFile, string sou`
`179`	`179`
`180`	`180`	`return list;`
`181`	`181`	`}`
`182`		`- catch (Exception ex)`
	`182`	`+ catch (Exception)`
`183`	`183`	`{`
`184`		`- Telemetry.TrackException(ex);`
`185`	`184`	`return null;`
`186`	`185`	`}`
`187`	`186`	`}`
`@@ -240,8 +239,6 @@ private CompilerResult ProcessConfig(string baseFolder, Config config)`
`240`	`239`	`OnAfterWritingSourceMap(absolute, mapFile, smChanges);`
`241`	`240`	`}`
`242`	`241`
`243`		`- Telemetry.TrackCompile(config);`
`244`		`-`
`245`	`242`	`return result;`
`246`	`243`	`}`
`247`	`244`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,6 @@ public override bool Execute()`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`Log.LogMessage(MessageImportance.High, Environment.NewLine + "WebCompiler: Begin compiling " + configFile.Name);`
`32`		`- Telemetry.SetDeviceName("MSBuild");`
`33`	`32`
`34`	`33`	`ConfigFileProcessor processor = new ConfigFileProcessor();`
`35`	`34`	`processor.BeforeProcess += (s, e) => { if (e.ContainsChanges) FileHelpers.RemoveReadonlyFlagFromFile(e.Config.GetAbsoluteOutputFile()); };`
`@@ -68,7 +67,6 @@ public override bool Execute()`
`68`	`67`	`catch (Exception ex)`
`69`	`68`	`{`
`70`	`69`	`Log.LogError(ex.Message);`
`71`		`- Telemetry.TrackException(ex);`
`72`	`70`	`return false;`
`73`	`71`	`}`
`74`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -19,8 +19,6 @@ static int Main(params string[] args)`
`19`	`19`	`return 0;`
`20`	`20`	`}`
`21`	`21`
`22`		`- Telemetry.SetDeviceName("Command line");`
`23`		`-`
`24`	`22`	`ConfigFileProcessor processor = new ConfigFileProcessor();`
`25`	`23`	`EventHookups(processor, configPath);`
`26`	`24`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,6 @@ public class BabelTest`
`13`	`13`	`[TestInitialize]`
`14`	`14`	`public void Setup()`
`15`	`15`	`{`
`16`		`- Telemetry.Enabled = false;`
`17`	`16`	`_processor = new ConfigFileProcessor();`
`18`	`17`	`}`
`19`	`18`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,6 @@ public class CoffeeScriptTest`
`13`	`13`	`[TestInitialize]`
`14`	`14`	`public void Setup()`
`15`	`15`	`{`
`16`		`- Telemetry.Enabled = false;`
`17`	`16`	`_processor = new ConfigFileProcessor();`
`18`	`17`	`}`
`19`	`18`