Merge branch 'main' into dev/gcampbell/FixIsUserPreset

.github/actions/.npmrc

@@ -0,0 +1,4 @@
+# [OSS Contributors] Delete this .npmrc file to use the npmjs registry instead of the Microsoft Azure Artifacts feed for local development
+
+registry=https://pkgs.dev.azure.com/azure-public/VisualCpp/_packaging/cpp_PublicPackages/npm/registry/                  
+always-auth=true

.npmrc

@@ -0,0 +1,4 @@
+# [OSS Contributors] Delete this .npmrc file to use the npmjs registry instead of the Microsoft Azure Artifacts feed for local development
+
+registry=https://pkgs.dev.azure.com/azure-public/VisualCpp/_packaging/cpp_PublicPackages/npm/registry/                  
+always-auth=true

CONTRIBUTING.md

@@ -34,3 +34,11 @@ Follow the [Typescript Coding guidelines](https://github.com/Microsoft/TypeScrip
 ### Changelog
 
 Please update the CHANGELOG.md file as part of your pull request. Follow the style within that file and give yourself credit for the changes you make.
+
+### Adding/Updating package.json dependencies
+
+We maintain a public Azure Artifacts feed that we point the package manager to in the .npmrc file.
+
+Because of this, if you want to add/update a dependency version in package.json, intervention from our team will be required to successfully merge PRs that have dependency updates. Please ping our team in the PR.
+
+For local development before attempting to merge a PR, you can delete the .npmrc file and use the default NPM feed directly. Please do not push any subsequent changes to the yarn.lock file.

jobs/release/prerelease.yml

@@ -29,9 +29,6 @@ extends:
         templateContext:
           type: releaseJob
           isProduction: true
-          sdl:
-            codeSignValidation:
-              additionalTargetsGlobPattern: -|**\*.js
           inputs:
           - input: pipelineArtifact
             pipeline: BuildPrereleasePipeline

jobs/release/release.yml

@@ -61,9 +61,6 @@ extends:
         templateContext:
           type: releaseJob
           isProduction: true
-          sdl:
-            codeSignValidation:
-              additionalTargetsGlobPattern: -|**\*.js
           inputs:
           - input: pipelineArtifact
             pipeline: BuildReleasePipeline

jobs/shared/build.yml

@@ -58,7 +58,7 @@ steps:
   displayName: Build files
   inputs:
     script: |
-      npm run vscode:prepublish
+      yarn compile-production
 - task: MSBuild@1
   displayName: Sign files
   inputs:

package.json

@@ -3677,7 +3677,6 @@
     "@test": "out/test/"
   },
   "scripts": {
-    "vscode:prepublish": "yarn run compile-production",
     "compile": "yarn install && webpack --mode development",
     "compile-watch": "yarn install && webpack --mode development --watch --progress",
     "compile-production": "yarn install && yarn run translations-generate && webpack --env BUILD_VSCODE_NLS=true --mode production",

tools/pr-creator/.npmrc

@@ -0,0 +1,4 @@
+# [OSS Contributors] Delete this .npmrc file to use the npmjs registry instead of the Microsoft Azure Artifacts feed for local development
+
+registry=https://pkgs.dev.azure.com/azure-public/VisualCpp/_packaging/cpp_PublicPackages/npm/registry/                  
+always-auth=true