WIP Donations block: Update donations block to use inner blocks

.eslintrc.normal.js

@@ -125,7 +125,7 @@ module.exports = {
 
 		// JSDoc plugin overrides
 		'jsdoc/check-alignment': 1, // Recommended
-		'jsdoc/check-examples': 1,
+		'jsdoc/check-examples': 0, // See https://github.com/eslint/eslint/issues/14745
 		'jsdoc/check-indentation': 1,
 		'jsdoc/check-param-names': 1, // Recommended
 		'jsdoc/check-syntax': 1,

.github/files/build-all-projects.sh

@@ -30,7 +30,7 @@ echo "::group::Changelogger setup"
 echo "::endgroup::"
 
 echo "::group::Determining build order"
-TMP="$(tools/get-build-order.php)"
+TMP="$(pnpx jetpack dependencies build-order --pretty --ignore-root)"
 SLUGS=()
 mapfile -t SLUGS <<<"$TMP"
 echo "::endgroup::"
@@ -79,7 +79,7 @@ for SLUG in "${SLUGS[@]}"; do
 			OLDLOCK=
 		fi
 	fi
-	if (cd $BASE && pnpx jetpack build "${SLUG}" -v --production); then
+	if (cd $BASE && pnpx jetpack build "${SLUG}" -v --production --timing --no-pnpm-install); then
 		FAIL=false
 	else
 		FAIL=true
@@ -190,7 +190,7 @@ for SLUG in "${SLUGS[@]}"; do
 		# Copy the resulting list of files into the clone.
 		xargs cp --parents --target-directory="$BUILD_DIR"
 
-	if [[ "$SLUG" == "plugins/jetpack" ]]; then
+	if [[ "$SLUG" == "plugins/jetpack" || "$SLUG" == "plugins/backup" ]]; then
 		echo "::group::Copying Jetpack files for backward compatibility."
 
 		OLD_VENDOR_DIR="$BUILD_DIR/vendor"

.github/files/create-e2e-projects-matrix.sh

@@ -0,0 +1,44 @@
+#!/bin/bash
+
+set -eo pipefail
+
+PROJECTS=('{"project":"Jetpack","path":"projects/plugins/jetpack/tests/e2e","testArgs":[],"slackArgs":[]}' '{"project":"Boost","path":"projects/plugins/boost/tests/e2e","testArgs":[],"slackArgs":[]}')
+PROJECTS_MATRIX=()
+RUN_NAME=''
+
+if [[ "$GITHUB_EVENT_NAME" == "pull_request" || "$GITHUB_EVENT_NAME" == "push" ]]; then
+	CHANGED_PROJECTS="$(.github/files/list-changed-projects.sh)"
+
+	for PROJECT in "${PROJECTS[@]}"; do
+		PROJECT_PATH=$(jq -r ".path" <<<"$PROJECT")
+		TARGETS=$(jq -r -e ".ci.targets" "$PROJECT_PATH/package.json")
+
+		if [[ "$TARGETS" == "[]" || "$TARGETS" == "" ]]; then
+			# if no target projects are found run the tests
+			PROJECTS_MATRIX+=("$PROJECT")
+		else
+			# iterate over defined target plugins/projects and see if they are changed
+			for TARGET in $(jq -r -e ".[]" <<<"$TARGETS"); do
+				RESULT=$(jq --arg prj "$TARGET" '.[$prj]' <<<"$CHANGED_PROJECTS")
+				if [[ "$RESULT" == true ]]; then
+					PROJECTS_MATRIX+=("$PROJECT")
+					break
+				fi
+			done
+		fi
+	done
+else
+	# gutenberg scheduled run
+	if [ "$CRON" == "0 */12 * * *" ]; then
+		PROJECTS_MATRIX+=('{"project":"Jetpack with Gutenberg","path":"projects/plugins/jetpack/tests/e2e","testArgs":["blocks"],"slackArgs":["--report", "gutenberg"]}')
+		RUN_NAME='gutenberg'
+	fi
+
+	# atomic scheduled run
+	if [ "$CRON" == "30 */4 * * *" ]; then
+		PROJECTS_MATRIX+=('{"project":"Jetpack on Atomic","path":"projects/plugins/jetpack/tests/e2e","testArgs":["blocks", "--grep-invert", "wordads"],"slackArgs":["--report", "atomic"]}')
+		RUN_NAME='atomic'
+	fi
+fi
+
+jq -n -c --arg runName "$RUN_NAME" --argjson projects "$(jq -s -c -r '.' <<<"${PROJECTS_MATRIX[@]}")" '{ "run": $runName, "matrix": $projects }'

.github/files/generate-ci-matrix.php

@@ -58,7 +58,7 @@
 		'script'  => 'test-php',
 		'php'     => $php,
 		'wp'      => 'latest',
-		'timeout' => 15, // 2021-01-18: Successful runs seem to take ~8 minutes for PHP 5.6 and for the 7.4 master run, ~5.5-6 for 7.x and 8.0.
+		'timeout' => 20, // 2022-01-25: 5.6 tests have started timing out at 15 minutes. Previously: Successful runs seem to take ~8 minutes for PHP 5.6 and for the 7.4 master run, ~5.5-6 for 7.x and 8.0.
 	);
 }
 // Merge this into the above once we decide PHP 8.1 is stable and WP latest works with 8.1.

.github/files/list-changed-projects.sh

@@ -0,0 +1,35 @@
+#!/bin/bash
+
+set -eo pipefail
+
+cd $(dirname "${BASH_SOURCE[0]}")/../..
+
+. tools/includes/chalk-lite.sh
+
+function debug {
+	blue "$@" >&2
+}
+
+function die {
+	echo "::error::$*" >&2
+	exit 1
+}
+
+ARGS=( --add-dependents )
+
+# Code coverage probably needs to run all tests to avoid confusing metrics.
+# @todo Switch it to use codecov.io's carryforward: https://about.codecov.io/blog/new-product-test-only-what-you-change-with-carryforward-flags/
+if [[ "$TEST_SCRIPT" == "test-coverage" ]]; then
+	debug "TEST_SCRIPT is test-coverage, considering all projects changed."
+elif [[ "${GITHUB_EVENT_NAME:?}" == "pull_request" ]]; then
+	[[ -f "${GITHUB_EVENT_PATH:?}" ]] || die "GITHUB_EVENT_PATH file $GITHUB_EVENT_PATH does not exist"
+	DIFF="$(jq -r '"\( .pull_request.base.sha )..\( .pull_request.head.sha )"' "$GITHUB_EVENT_PATH")"
+	debug "GITHUB_EVENT_NAME is pull_request, checking diff from $DIFF"
+	ARGS+=( --verbose "--git-changed=$DIFF" )
+elif [[ "${GITHUB_EVENT_NAME:?}" == "push" ]]; then
+	debug "GITHUB_EVENT_NAME is push, considering all projects changed."
+else
+	die "Unsupported GITHUB_EVENT_NAME \"$GITHUB_EVENT_NAME\""
+fi
+
+pnpx jetpack dependencies list "${ARGS[@]}" | jq -ncR 'reduce inputs as $i ({}; .[$i] |= true)'

.github/files/required-review.yaml

@@ -22,6 +22,8 @@
   paths:
    - 'projects/packages/**'
    - 'projects/plugins/jetpack/**'
+   # Exclude packages managed by other teams, which are covered by other blocks below.
+   - '!projects/packages/search/**'
   teams:
    - jetpack-approvers
 
@@ -44,10 +46,11 @@
    - heart-of-gold
    - jetpack-approvers
 
-# The Search team reviews changes to the Search plugin
+# The Search team reviews changes to the Search plugin, and the Search package.
 - name: Search
   paths:
    - 'projects/plugins/search/**'
+   - 'projects/packages/search/**'
   teams:
    - jetpack-search
    - jetpack-approvers

.github/files/setup-wordpress-env.sh

@@ -35,12 +35,6 @@ case "$WP_BRANCH" in
 		;;
 	latest)
 		LATEST=$(php ./tools/get-wp-version.php)
-		# 5.8.x still requires a monkey-patched obsolete version of phpunit.
-		# If that's latest, run the coverage test with the upcoming 5.9 instead.
-		# @todo: Remove this once WordPress 5.9 is "latest".
-		if [[ "$LATEST" == 5.8.* && "$TEST_SCRIPT" == "test-coverage" ]]; then
-			LATEST=master
-		fi
 		git clone --depth=1 --branch "$LATEST" git://develop.git.wordpress.org/ /tmp/wordpress-latest
 		;;
 	previous)