Merge branch 'trunk' into task/22721-stats-traffic-improve-scrolling-performance

Author: staskus

.buildkite/pipeline.yml

@@ -1,12 +1,5 @@
-# Nodes with values to reuse in the pipeline.
-common_params:
-  # Common plugin settings to use with the `plugins` key.
-  - &common_plugins
-    - automattic/a8c-ci-toolkit#3.1.0
-  # Common environment values to use with the `env` key.
-  - &common_env
-    # Be sure to also update the `.xcode-version` file when updating the Xcode image/version here
-    IMAGE_ID: xcode-15.1
+env:
+  IMAGE_ID: $IMAGE_ID
 
 # This is the default pipeline – it will build and test the app
 steps:
@@ -18,17 +11,15 @@ steps:
     steps:
       - label: "🛠 WordPress Prototype Build"
         command: ".buildkite/commands/prototype-build-wordpress.sh"
-        env: *common_env
-        plugins: *common_plugins
+        plugins: [$CI_TOOLKIT_PLUGIN]
         if: "build.pull_request.id != null || build.pull_request.draft"
         notify:
           - github_commit_status:
               context: "WordPress Prototype Build"
 
       - label: "🛠 Jetpack Prototype Build"
         command: ".buildkite/commands/prototype-build-jetpack.sh"
-        env: *common_env
-        plugins: *common_plugins
+        plugins: [$CI_TOOLKIT_PLUGIN]
         if: "build.pull_request.id != null || build.pull_request.draft"
         notify:
           - github_commit_status:
@@ -42,17 +33,15 @@ steps:
       - label: "🛠 :wordpress: Build for Testing"
         key: "build_wordpress"
         command: ".buildkite/commands/build-for-testing.sh wordpress"
-        env: *common_env
-        plugins: *common_plugins
+        plugins: [$CI_TOOLKIT_PLUGIN]
         notify:
           - github_commit_status:
               context: "WordPress Build for Testing"
 
       - label: "🛠 :jetpack: Build for Testing"
         key: "build_jetpack"
         command: ".buildkite/commands/build-for-testing.sh jetpack"
-        env: *common_env
-        plugins: *common_plugins
+        plugins: [$CI_TOOLKIT_PLUGIN]
         notify:
           - github_commit_status:
               context: "Jetpack Build for Testing"
@@ -63,8 +52,7 @@ steps:
   - label: "🔬 :wordpress: Unit Tests"
     command: ".buildkite/commands/run-unit-tests.sh"
     depends_on: "build_wordpress"
-    env: *common_env
-    plugins: *common_plugins
+    plugins: [$CI_TOOLKIT_PLUGIN]
     artifact_paths:
       - "build/results/*"
     notify:
@@ -79,8 +67,7 @@ steps:
       - label: "🔬 :jetpack: UI Tests (iPhone)"
         command: .buildkite/commands/run-ui-tests.sh 'iPhone 15'
         depends_on: "build_jetpack"
-        env: *common_env
-        plugins: *common_plugins
+        plugins: [$CI_TOOLKIT_PLUGIN]
         artifact_paths:
           - "build/results/*"
           - "build/results/crashes/*"
@@ -89,10 +76,9 @@ steps:
               context: "UI Tests (iPhone)"
 
       - label: "🔬 :jetpack: UI Tests (iPad)"
-        command: .buildkite/commands/run-ui-tests.sh 'iPad Pro (12.9-inch) (6th generation)'
+        command: .buildkite/commands/run-ui-tests.sh 'iPad Pro 13-inch (M4)'
         depends_on: "build_jetpack"
-        env: *common_env
-        plugins: *common_plugins
+        plugins: [$CI_TOOLKIT_PLUGIN]
         artifact_paths:
           - "build/results/*"
           - "build/results/crashes/*"
@@ -105,14 +91,23 @@ steps:
   #################
   - group: "Linters"
     steps:
+      - label: "☢️ Danger - PR Check"
+        command: danger
+        key: danger
+        if: "build.pull_request.id != null"
+        retry:
+          manual:
+            permit_on_passed: true
+        agents:
+          queue: "linter"
+
       - label: ":swift: SwiftLint"
-        command: run_swiftlint --strict
-        plugins: *common_plugins
+        command: swiftlint
         notify:
           - github_commit_status:
               context: "SwiftLint"
         agents:
-          queue: "default"
+          queue: "linter"
 
       - label: "🧹 Lint Translations"
         command: "gplint /workdir/WordPress/Resources/AppStoreStrings.po"
@@ -127,5 +122,4 @@ steps:
 
       - label: ":sleuth_or_spy: Lint Localized Strings Format"
         command: .buildkite/commands/lint-localized-strings-format.sh
-        plugins: *common_plugins
-        env: *common_env
+        plugins: [$CI_TOOLKIT_PLUGIN]

.buildkite/release-builds.yml

@@ -1,24 +1,18 @@
 # This pipeline is meant to be run via the Buildkite API, and is only used for release builds
 
-# Nodes with values to reuse in the pipeline.
-common_params:
-  # Common plugin settings to use with the `plugins` key.
-  - &common_plugins
-    - automattic/a8c-ci-toolkit#3.1.0
-
 env:
-  IMAGE_ID: xcode-15.1
+  IMAGE_ID: $IMAGE_ID
 
 steps:
 
   - label: ":wordpress: :testflight: WordPress Release Build (App Store Connect)"
     command: ".buildkite/commands/release-build-wordpress.sh $BETA_RELEASE"
-    plugins: *common_plugins
+    plugins: [$CI_TOOLKIT_PLUGIN]
     notify:
     - slack: "#build-and-ship"
 
   - label: ":jetpack: :testflight: Jetpack Release Build (App Store Connect)"
     command: ".buildkite/commands/release-build-jetpack.sh"
-    plugins: *common_plugins
+    plugins: [$CI_TOOLKIT_PLUGIN]
     notify:
     - slack: "#build-and-ship"

.buildkite/release-pipelines/code-freeze.yml

@@ -1,7 +1,7 @@
 steps:
   - label: Code Freeze
     plugins:
-      - automattic/a8c-ci-toolkit#3.1.0
+      - $CI_TOOLKIT_PLUGIN
     # The first client to implement releases in CI was Android so the automation works in that queue.
     # We might want to move it to a leaner one in the future.
     agents:

.buildkite/release-pipelines/complete-code-freeze.yml

@@ -1,26 +1,17 @@
-common_params:
-  # Common plugin settings to use with the `plugins` key.
-  - &common_plugins
-    - automattic/a8c-ci-toolkit#3.1.0
-  # Common environment values to use with the `env` key.
-  - &common_env
-    # Be sure to also update the `.xcode-version` file when updating the Xcode image/version here
-    IMAGE_ID: xcode-15.1
+env:
+  IMAGE_ID: $IMAGE_ID
+
+# The code freeze completion needs to run on macOS because it uses genstrings under the hood
+agents:
+    queue: mac
 
 steps:
   - label: Complete Code Freeze
     key: complete_code_freeze
-    plugins: *common_plugins
-    env: *common_env
-    # The code freeze completion needs to run on macOS because it uses genstrings under the hood
-    agents:
-        queue: mac
+    plugins: [$CI_TOOLKIT_PLUGIN]
     command: ".buildkite/commands/complete-code-freeze.sh $RELEASE_VERSION"
 
   - label: Log Outdated Pods
     depends_on: complete_code_freeze
-    plugins: *common_plugins
-    env: *common_env
-    agents:
-        queue: mac
+    plugins: [$CI_TOOLKIT_PLUGIN]
     command: ".buildkite/commands/log-outdated-pods.sh $RELEASE_VERSION"

.buildkite/release-pipelines/finalize-hotfix.yml

@@ -1,10 +1,10 @@
 steps:
   - label: Finalize Release
     plugins:
-      - automattic/a8c-ci-toolkit#3.1.0
+      - $CI_TOOLKIT_PLUGIN
     # The finalization needs to run on macOS because of localization linting
     agents:
         queue: mac
     env:
-      IMAGE_ID: xcode-15.1
+      IMAGE_ID: $IMAGE_ID
     command: ".buildkite/commands/finalize-hotfix.sh $VERSION"

.buildkite/release-pipelines/finalize-release.yml

@@ -1,10 +1,10 @@
 steps:
   - label: Finalize Release
     plugins:
-      - automattic/a8c-ci-toolkit#3.1.0
+      - $CI_TOOLKIT_PLUGIN
     # The finalization needs to run on macOS because of localization linting
     agents:
         queue: mac
     env:
-      IMAGE_ID: xcode-15.1
+      IMAGE_ID: $IMAGE_ID
     command: ".buildkite/commands/finalize-release.sh $RELEASE_VERSION"

.buildkite/release-pipelines/new-beta-release.yml

@@ -1,12 +1,12 @@
 steps:
   - label: New Beta Deployment
     plugins:
-      - automattic/a8c-ci-toolkit#3.1.0
+      - $CI_TOOLKIT_PLUGIN
     # The beta needs to run on macOS because it uses genstrings under the hood
     agents:
         queue: mac
     env:
-      IMAGE_ID: xcode-15.1
+      IMAGE_ID: $IMAGE_ID
     command: |
       echo '--- :git: Configure Git for release management'
       .buildkite/commands/configure-git-for-release-management.sh

.buildkite/release-pipelines/new-hotfix.yml

@@ -1,12 +1,12 @@
 steps:
   - label: New Hotfix Deployment
     plugins:
-      - automattic/a8c-ci-toolkit#3.1.0
-    # The beta needs to run on macOS because it uses genstrings under the hood
+      - $CI_TOOLKIT_PLUGIN
+    # The hotfix needs to run on macOS because it uses genstrings under the hood
     agents:
         queue: mac
     env:
-      IMAGE_ID: xcode-15.1
+      IMAGE_ID: $IMAGE_ID
     command: |
       echo '--- :git: Configure Git for release management'
       .buildkite/commands/configure-git-for-release-management.sh

.buildkite/release-pipelines/update-app-store-strings.yml

@@ -1,7 +1,7 @@
 steps:
   - label: Update App Store Strings
     plugins:
-      - automattic/a8c-ci-toolkit#3.1.0
+      - $CI_TOOLKIT_PLUGIN
     # The first client to implement releases in CI was Android so the automation works in that queue.
     # We might want to move it to a leaner one in the future.
     agents:

.buildkite/shared-pipeline-vars

@@ -0,0 +1,10 @@
+#!/bin/sh
+
+# This file is `source`'d before calling `buildkite-agent pipeline upload`, and can be used
+# to set up some variables that will be interpolated in the `.yml` pipeline before uploading it.
+
+# The ~> modifier is not currently used, but we check for it just in case
+XCODE_VERSION=$(sed -E -n 's/^(~> )?(.*)/xcode-\2/p' .xcode-version)
+export IMAGE_ID="$XCODE_VERSION"
+
+export CI_TOOLKIT_PLUGIN="automattic/a8c-ci-toolkit#3.3.0"