Merge branch 'main' into snippet-foreach-progress

Author: ian-mor

.config/tsaoptions.json

@@ -0,0 +1,8 @@
+{
+  "instanceUrl": "https://msazure.visualstudio.com",
+  "projectName": "One",
+  "areaPath": "One\\MGMT\\Compute\\Powershell\\Powershell",
+  "notificationAliases": [ "[email protected]", "[email protected]" ],
+  "codebaseName": "PowerShell_vscode-powershell_20240328",
+  "tools": [ "CredScan", "PoliCheck", "BinSkim" ]
+}

.eslintrc.json

@@ -33,42 +33,28 @@
       "error",
       "always"
     ],
-    "@typescript-eslint/explicit-function-return-type": [
-      "error"
-    ],
+    "@typescript-eslint/explicit-function-return-type": "error",
+    "@typescript-eslint/no-empty-object-type": "off",
     "@typescript-eslint/no-floating-promises": [
       "error",
       {
         "ignoreVoid": true
       }
     ],
-    "@typescript-eslint/no-non-null-assertion": [
-      "off"
-    ],
+    "@typescript-eslint/no-non-null-assertion": "off",
+    "@typescript-eslint/no-require-imports": "off",
     "@typescript-eslint/no-unused-vars": [
       "error",
       {
         "argsIgnorePattern": "^_"
       }
     ],
-    "@typescript-eslint/no-unsafe-argument": [
-      "off"
-    ],
-    "@typescript-eslint/no-unsafe-assignment": [
-      "off"
-    ],
-    "@typescript-eslint/no-unsafe-call": [
-      "off"
-    ],
-    "@typescript-eslint/no-unsafe-member-access": [
-      "off"
-    ],
-    "@typescript-eslint/no-unsafe-return": [
-      "off"
-    ],
-    "@typescript-eslint/restrict-template-expressions": [
-      "off"
-    ],
+    "@typescript-eslint/no-unsafe-argument": "off",
+    "@typescript-eslint/no-unsafe-assignment": "off",
+    "@typescript-eslint/no-unsafe-call": "off",
+    "@typescript-eslint/no-unsafe-member-access": "off",
+    "@typescript-eslint/no-unsafe-return": "off",
+    "@typescript-eslint/restrict-template-expressions": "off",
     "@typescript-eslint/prefer-nullish-coalescing": [
       "error",
       {

.github/dependabot.yml

@@ -0,0 +1,16 @@
+changelog:
+  exclude:
+    labels:
+      - Ignore
+    authors:
+      - dependabot
+  categories:
+    - title: Enhancements & Features ✨
+      labels:
+        - Issue-Enhancement
+    - title: Squashed Bugs 🐛
+      labels:
+        - Issue-Bug
+    - title: Other Changes 🙏
+      labels:
+        - "*"

.github/release.yml

@@ -6,7 +6,6 @@ on:
   pull_request:
     # The branches below must be a subset of the branches above
     branches: [ main ]
-    paths-ignore: [ '**/*.md' ]
   merge_group:
     types: [ checks_requested ]
 
@@ -19,12 +18,8 @@ jobs:
     runs-on: ${{ matrix.os }}
     env:
       DOTNET_NOLOGO: true
-      DOTNET_CLI_TELEMETRY_OPTOUT: true
       DOTNET_GENERATE_ASPNET_CERTIFICATE: false
       DISPLAY: ':99.0'
-    defaults:
-      run:
-        working-directory: vscode-powershell
     steps:
       - name: Checkout PowerShellEditorServices
         uses: actions/checkout@v4
@@ -41,16 +36,17 @@ jobs:
         uses: actions/setup-dotnet@v4
         with:
           cache: true
-          cache-dependency-path: 'PowerShellEditorServices/**/packages.lock.json'
+          cache-dependency-path: PowerShellEditorServices/**/*.csproj
           global-json-file: PowerShellEditorServices/global.json
+          config-file: PowerShellEditorServices/nuget.config
 
       - name: Install PSResources
         shell: pwsh
-        run: ../PowerShellEditorServices/tools/installPSResources.ps1
+        run: ./vscode-powershell/tools/installPSResources.ps1
 
       - uses: actions/setup-node@v4
         with:
-          node-version: 18
+          node-version: 20
           cache: npm
           cache-dependency-path: vscode-powershell/package-lock.json
 
@@ -60,14 +56,15 @@ jobs:
 
       - name: Build, test and package
         shell: pwsh
-        run: Invoke-Build
+        run: Invoke-Build -Configuration Release
+        working-directory: vscode-powershell
 
       - name: Upload build artifacts
         uses: actions/upload-artifact@v4
         if: always()
         with:
           name: vscode-powershell-vsix-${{ matrix.os }}
-          path: vscode-powershell/powershell-*.vsix
+          path: '**/*.vsix'
 
       - name: Upload test results
         uses: actions/upload-artifact@v4

.github/workflows/ci-test.yml

@@ -1,12 +1,13 @@
+**.js
+**.js.map
 .vscode-test/
-logs/
+*.vsix
+dist/
 modules
 modules/
 node_modules/
-obj/
-bin/
 out/
-sessions/
+test/mocks/BinaryModule/bin/
+test/mocks/BinaryModule/obj/
 test-results.xml
-*.vsix
-*.DS_Store
+tsconfig.tsbuildinfo

.github/workflows/codeql-analysis.yml

@@ -8,5 +8,5 @@
   "require": "source-map-support/register",
   "timeout": 600000,
   "slow": 2000,
-  "spec": "out/test/**/*.test.js"
+  "spec": "test/**/*.test.js"
 }

.gitignore

@@ -1,2 +1,7 @@
-; We use Dependabot to update our packages, so we want to add without a prefix
-save-exact=true
+; We generally want to save install/update commands
+save=true
+; We use a public Azure Artifacts mirror
+registry=https://pkgs.dev.azure.com/powershell/PowerShell/_packaging/powershell/npm/registry/
+always-auth=true
+; But we don't want references to it in the lockfile
+omit-lockfile-registry-resolved=true