Skip to content

Remove @microsoft/applicationinsights-web dependency

Paul Slaughter requested to merge ps-remove-appinsights-package into main
  • Also replace WebAppInsightsAppender with NullAppender

How to test?

You can update the vscode_version.json of the gitlab-web-ide project to use the build artifact from this MR.

diff --git a/packages/vscode-build/vscode_version.json b/packages/vscode-build/vscode_version.json
index 2d8b458..bacfd79 100644
--- a/packages/vscode-build/vscode_version.json
+++ b/packages/vscode-build/vscode_version.json
@@ -1,5 +1,5 @@
 {
   "type": "url",
-  "location": "https://gitlab.com/gitlab-org/gitlab-web-ide-vscode-fork/-/package_files/47251282/download",
-  "version": "1.69.1-1.0.0-dev-20220725055226"
+  "location": "https://gitlab.com/gitlab-org/gitlab-web-ide-vscode-fork/-/jobs/2945469975/artifacts/raw/.build/vscode-web-dist/vscode-web-1.69.1-1.0.0-dev-20220829034959.tar.gz",
+  "version": "1.69.1-1.0.0-dev-20220829034959"
 }

Screenshots

This results in about 50M of uncompressed package size reduction (203M vs. 155M), which ends up being about 7M of compressed package size reduction.

Screen_Shot_2022-08-28_at_10.59.07_PM

Screen_Shot_2022-08-28_at_10.56.24_PM

When hooking this MR's build artifact into the web-ide, it looks like everything still works and will save more than 10M of uncompressed package size 👍

Screen_Shot_2022-08-28_at_11.14.47_PM

Screen_Shot_2022-08-28_at_11.24.33_PM

Context

gitlab-web-ide!51 (closed)

Edited by Paul Slaughter

Merge request reports