[pull] master from microsoft:master

.devops/templates/publish-website.yml

@@ -1,7 +1,5 @@
-parameters:
-  - name: version
-    type: string
-    default: 8
+# This template is no longer used in the v8 release pipeline, since v8 website updates are unlikely to be needed.
+# If this is re-enabled, the Azure service connection will need to be re-created.
 
 steps:
   - task: AzureCLI@2
@@ -15,7 +13,7 @@ steps:
         --destination 'fabric-website' \
         --source $(System.DefaultWorkingDirectory)/apps/public-docsite/dist \
         --account-name fabricweb \
-        --destination-path v${{ parameters.version }}/$(Build.BuildNumber)/ \
+        --destination-path v8/$(Build.BuildNumber)/ \
         --auth-mode login \
         --overwrite
 
@@ -29,8 +27,8 @@ steps:
         az storage blob upload \
         --account-name fabricweb \
         --container-name fabric-website \
-        --name manifests/v${{ parameters.version }}-df.js \
-        --file $(System.DefaultWorkingDirectory)/site-manifests/v${{ parameters.version }}-df.js \
+        --name manifests/v8-df.js \
+        --file $(System.DefaultWorkingDirectory)/site-manifests/v8-df.js \
         --auth-mode login \
         --overwrite
 
@@ -44,7 +42,7 @@ steps:
         az storage blob upload \
         --account-name fabricweb \
         --container-name fabric-website \
-        --name manifests/v${{ parameters.version }}-prod.js \
-        --file $(System.DefaultWorkingDirectory)/site-manifests/v${{ parameters.version }}-prod.js \
+        --name manifests/v8-prod.js \
+        --file $(System.DefaultWorkingDirectory)/site-manifests/v8-prod.js \
         --auth-mode login \
         --overwrite

.devops/templates/tools.yml

@@ -1,9 +1,13 @@
-# Install versions of Node and Yarn required by build pipelines.
+# Install tools and dependencies and set shared configuration
 parameters:
   - name: dryRun
     displayName: Dry Run Mode
     type: boolean
     default: false
+  - name: githubPAT
+    displayName: GitHub Personal Access Token
+    type: string
+    default: ''
 
 steps:
   # Logs a message when dry run mode is enabled.
@@ -25,9 +29,6 @@ steps:
     displayName: 'Install Node.js'
     retryCountOnTaskFailure: 1
 
-  - script: corepack enable
-    displayName: 'Enable Corepack'
-
   - script: |
       echo "nodejs:" && node -v
       echo "yarn:" && yarn -v
@@ -48,10 +49,21 @@ steps:
   - script: |
       printenv | sort
       echo "SHELLOPTS $SHELLOPTS"
-      echo 'deployBasePath "$(deployBasePath)"'
-      echo 'deployUrl "$(deployUrl)"'
-      echo 'isPR "$(isPR)"'
-      echo 'targetBranch "$(targetBranch)"'
       echo number of CPUs "$(getconf _NPROCESSORS_ONLN)"
     displayName: Log environment variables (Linux)
     condition: eq(variables['Agent.OS'], 'Linux')
+
+  - script: |
+      git config user.name "Fluent UI Build"
+      git config user.email "fluentui-internal@service.microsoft.com"
+    displayName: Configure git user (used by beachball)
+
+  - script: |
+      git remote set-url origin https://fabricteam:$(githubPAT)@github.com/microsoft/fluentui.git
+    displayName: Authenticate git for pushes
+    condition: and(succeeded(), ne('${{ parameters.githubPAT }}', ''))
+
+  - script: |
+      corepack enable
+      yarn install --immutable
+    displayName: yarn install

.devops/templates/variables.yml

@@ -1,45 +1,7 @@
-parameters:
-  # For customizing the deployment path in non-PR builds
-  - name: deployBasePath
-    type: string
-    default: ''
-
-  # Skip the component governance detection step (injected by a pipeline decorator from an
-  # internal extension) by default because we run it separately. Since all our pipelines
-  # in each branch install the same packages, only one pipeline (currently the daily release)
-  # needs to run detection.
-  - name: skipComponentGovernanceDetection
-    type: boolean
-    default: true
-
 variables:
   # Prevents failures on CI when "caniuse-lite" becomes outdated
   BROWSERSLIST_IGNORE_OLD_DATA: true
 
   NX_PARALLEL: 8
   NX_PREFER_TS_NODE: true
   NX_VERBOSE_LOGGING: true
-
-  # Also accessed as process.env.DEPLOYHOST
-  deployHost: 'fluentuipr.z22.web.core.windows.net'
-
-  # Also accessed as process.env.DEPLOYURL
-  deployUrl: 'https://$(deployHost)/$(deployBasePath)'
-
-  # This service principal ("subscription" is a misleading name) only has access to the fluentuipr storage account
-  azureSubscription: Azure PR deploy - NEW
-  azureStorage: fluentuipr
-
-  ${{ if not(startsWith(variables['Build.SourceBranch'], 'refs/heads/')) }}:
-    isPR: true
-    targetBranch: 'origin/$(System.PullRequest.TargetBranch)'
-    # Deploy PRs under "pull/####" unless otherwise requested
-    # (this is also accessed as process.env.DEPLOYBASEPATH)
-    deployBasePath: ${{ coalesce(parameters.deployBasePath, 'pull/$(System.PullRequest.PullRequestNumber)') }}
-  ${{ if startsWith(variables['Build.SourceBranch'], 'refs/heads/') }}:
-    isPR: false
-    targetBranch: ''
-    # Deploy master under "heads/branchname" unless otherwise requested
-    deployBasePath: ${{ coalesce(parameters.deployBasePath, replace(variables['Build.SourceBranch'], 'refs/', '')) }}
-
-  skipComponentGovernanceDetection: ${{ parameters.skipComponentGovernanceDetection }}

.github/workflows/pr.yml

@@ -58,10 +58,9 @@ jobs:
         run: |
           # following packages need to be build in advance:
           # @fluentui/api-docs is used within apps/public-docsite-resources/just.config.ts,
-          # @fluentui/digest is used within packages/fluentui/perf-test-northstar/just.config.ts, thus it needs to be build in advance
           #
           # NOTE: we are running this via nx in order to get cache hits later on
-          yarn nx run-many -t build -p api-docs digest
+          yarn nx run-many -t build -p api-docs
           yarn tsc -p ./tsconfig.just-scripts-configs.json
 
       - name: 'check packages: installed dependencies versions'

.vscode/launch.json

@@ -35,24 +35,6 @@
       "outputCapture": "std",
       "console": "integratedTerminal"
     },
-    {
-      "name": "Debug current open test (v0)",
-      "type": "node",
-      "request": "launch",
-      "program": "${workspaceRoot}/scripts/executors/src/debug-test.js",
-      "cwd": "${fileDirname}",
-      "stopOnEntry": false,
-      "args": ["-i", "--testPathPatterns=\\b${fileBasenameNoExtension}", "--watch"],
-      "runtimeExecutable": null,
-      "runtimeArgs": ["--nolazy", "--inspect"],
-      "env": {
-        "NODE_ENV": "test"
-      },
-      "sourceMaps": true,
-      "outFiles": ["${workspaceRoot}/packages/fluentui/*/dist/**/*.js"],
-      "outputCapture": "std",
-      "console": "integratedTerminal"
-    },
     {
       "name": "Build @fluentui/react",
       "type": "node",
@@ -160,24 +142,6 @@
       "sourceMaps": true,
       "console": "integratedTerminal"
     },
-    {
-      "name": "Debug gulp task",
-      "type": "node",
-      "request": "launch",
-      "program": "${workspaceRoot}/node_modules/gulp/bin/gulp.js",
-      "stopOnEntry": false,
-      "runtimeArgs": ["--nolazy", "--inspect"],
-      // You can change the task name and cwd locally as needed
-      "args": ["build"],
-      "cwd": "${workspaceRoot}/packages/fluentui/react-northstar",
-      "env": {
-        "NODE_ENV": "development",
-        // This is used in scripts/babel/index.js to enable sourcemaps
-        "DEBUG": "1"
-      },
-      "sourceMaps": true,
-      "console": "integratedTerminal"
-    },
     {
       "name": "Debug create-package",
       "type": "node-terminal",

.vscode/settings.json

@@ -35,7 +35,13 @@
     ".yarn/releases": true,
     "common/temp": true,
     "**/node_modules": true,
-    "**/lib": true,
+    // VS Code doesn't support un-excluding things, so this is the only workaround for some nx projects using "lib" for source
+    "apps/*/lib": true,
+    "packages/*/lib": true,
+    "packages/react-components/*/lib": true,
+    "packages/react-components/*/*/lib": true,
+    "packages/charts/*/lib": true,
+    "packages/charts/*/*/lib": true,
     "**/lib-amd": true,
     "**/lib-commonjs": true,
     "**/dist": true,