Skip to content

Commit 32bca45

Browse files
committed
Build/Test Tools: Adjust various jobs names in the GitHub Actions workflows to remove duplication and improve grouping in the UI when they run.
Props desrosj, mukesh27, johnbillion See #63170 git-svn-id: https://develop.svn.wordpress.org/trunk@60628 602fd350-edb4-49c9-b593-d223f7449a82
1 parent 83e048b commit 32bca45

10 files changed

+21
-13
lines changed

.github/workflows/coding-standards.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,15 +50,15 @@ permissions: {}
5050
jobs:
5151
# Runs the PHP coding standards checks.
5252
phpcs:
53-
name: PHP coding standards
53+
name: Coding standards
5454
uses: ./.github/workflows/reusable-coding-standards-php.yml
5555
permissions:
5656
contents: read
5757
if: ${{ github.repository == 'WordPress/wordpress-develop' || ( github.event_name == 'pull_request' && github.actor != 'dependabot[bot]' ) }}
5858

5959
# Runs the JavaScript coding standards checks.
6060
jshint:
61-
name: JavaScript coding standards
61+
name: Coding standards
6262
uses: ./.github/workflows/reusable-coding-standards-javascript.yml
6363
permissions:
6464
contents: read

.github/workflows/end-to-end-tests.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ env:
5656
jobs:
5757
# Runs the end-to-end test suite.
5858
e2e-tests:
59-
name: Test with SCRIPT_DEBUG ${{ matrix.LOCAL_SCRIPT_DEBUG && 'enabled' || 'disabled' }}
59+
name: ${{ matrix.label }}
6060
uses: ./.github/workflows/reusable-end-to-end-tests.yml
6161
permissions:
6262
contents: read
@@ -65,6 +65,8 @@ jobs:
6565
fail-fast: false
6666
matrix:
6767
LOCAL_SCRIPT_DEBUG: [ true, false ]
68+
# A matrix value is needed in the 'name' directive for proper grouping in the GitHub UI.
69+
label: [ 'E2E Tests' ]
6870
with:
6971
LOCAL_SCRIPT_DEBUG: ${{ matrix.LOCAL_SCRIPT_DEBUG }}
7072

.github/workflows/performance.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,7 @@ jobs:
121121
matrix:
122122
memcached: [ true, false ]
123123
multisite: [ true, false ]
124+
# A matrix value is needed in the 'name' directive for proper grouping in the GitHub UI.
124125
label: [ Compare ]
125126
with:
126127
memcached: ${{ matrix.memcached }}

.github/workflows/phpunit-tests.yml

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -224,12 +224,15 @@ jobs:
224224
report: ${{ false }}
225225

226226
#
227-
# Runs specific individual test groups.
227+
# Runs the HTML API test group.
228+
#
229+
# This test group runs separately due to the large number of tests that are skipped in this group while the
230+
# HTML API is being developed. The skipped tests would otherwise cloud the results of all other test groups.
228231
#
229232
# These tests are run against the most recent LTS version of MySQL.
230233
#
231-
specific-test-groups:
232-
name: ${{ matrix.phpunit-test-groups }}
234+
html-api-test-groups:
235+
name: ${{ matrix.label }}
233236
uses: ./.github/workflows/reusable-phpunit-tests-v3.yml
234237
permissions:
235238
contents: read
@@ -242,6 +245,8 @@ jobs:
242245
db-type: [ 'mysql' ]
243246
db-version: [ '8.4' ]
244247
phpunit-test-groups: [ 'html-api-html5lib-tests' ]
248+
# A matrix value is needed in the 'name' directive for proper grouping in the GitHub UI.
249+
label: [ 'HTML API' ]
245250
with:
246251
php: ${{ matrix.php }}
247252
db-type: ${{ matrix.db-type }}
@@ -312,7 +317,7 @@ jobs:
312317
permissions:
313318
actions: read
314319
contents: read
315-
needs: [ test-with-mysql, test-with-mariadb, test-innovation-releases, specific-test-groups, limited-matrix-for-forks ]
320+
needs: [ test-with-mysql, test-with-mariadb, test-innovation-releases, html-api-test-groups, limited-matrix-for-forks ]
316321
if: ${{ github.repository == 'WordPress/wordpress-develop' && github.event_name != 'pull_request' && always() }}
317322
with:
318323
calling_status: ${{ contains( needs.*.result, 'cancelled' ) && 'cancelled' || contains( needs.*.result, 'failure' ) && 'failure' || 'success' }}

.github/workflows/reusable-coding-standards-javascript.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
# - Run the WordPress JSHint checks.
2727
# - Ensures version-controlled files are not modified or deleted.
2828
jshint:
29-
name: Run coding standards checks
29+
name: JavaScript checks
3030
runs-on: ubuntu-24.04
3131
permissions:
3232
contents: read

.github/workflows/reusable-coding-standards-php.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jobs:
3838
# - Generate a report for displaying `test` directory issues as pull request annotations.
3939
# - Ensures version-controlled files are not modified or deleted.
4040
phpcs:
41-
name: Run coding standards checks
41+
name: PHP checks
4242
runs-on: ubuntu-24.04
4343
permissions:
4444
contents: read

.github/workflows/reusable-end-to-end-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ jobs:
6363
# - Uploads screenshots and HTML snapshots as an artifact.
6464
# - Ensures version-controlled files are not modified or deleted.
6565
e2e-tests:
66-
name: Run E2E tests
66+
name: SCRIPT_DEBUG ${{ inputs.LOCAL_SCRIPT_DEBUG && 'enabled' || 'disabled' }}
6767
runs-on: ubuntu-24.04
6868
permissions:
6969
contents: read

.github/workflows/reusable-phpunit-tests-v3.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ jobs:
118118
# - Checks out the WordPress Test reporter repository.
119119
# - Submit the test results to the WordPress.org host test results.
120120
phpunit-tests:
121-
name: ${{ inputs.phpunit-test-groups && format( '{0} / ', inputs.phpunit-test-groups ) || '' }}PHP ${{ inputs.php }} ${{ ! inputs.phpunit-test-groups && ! inputs.coverage-report && '/ ' || 'with ' }}${{ 'mariadb' == inputs.db-type && 'MariaDB' || 'MySQL' }} ${{ inputs.db-version }}${{ inputs.multisite && ' multisite' || '' }}${{ inputs.db-innovation && ' (innovation release)' || '' }}${{ inputs.memcached && ' with memcached' || '' }}${{ inputs.report && ' (test reporting enabled)' || '' }} ${{ 'example.org' != inputs.tests-domain && inputs.tests-domain || '' }}
121+
name: ${{ ( inputs.phpunit-test-groups || inputs.coverage-report ) && format( 'PHP {0} with ', inputs.php ) || '' }} ${{ 'mariadb' == inputs.db-type && 'MariaDB' || 'MySQL' }} ${{ inputs.db-version }}${{ inputs.multisite && ' multisite' || '' }}${{ inputs.db-innovation && ' (innovation release)' || '' }}${{ inputs.memcached && ' with memcached' || '' }}${{ inputs.report && ' (test reporting enabled)' || '' }} ${{ 'example.org' != inputs.tests-domain && inputs.tests-domain || '' }}
122122
runs-on: ${{ inputs.os }}
123123
timeout-minutes: ${{ inputs.coverage-report && 120 || inputs.php == '8.4' && 30 || 20 }}
124124
permissions:

.github/workflows/reusable-test-core-build-process.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ jobs:
5656
# - Saves the pull request number to a text file.
5757
# - Uploads the pull request number as an artifact.
5858
build-process-tests:
59-
name: Core running from ${{ inputs.directory }} / ${{ contains( inputs.os, 'macos-' ) && 'MacOS' || contains( inputs.os, 'windows-' ) && 'Windows' || 'Linux' }}
59+
name: ${{ contains( inputs.os, 'macos-' ) && 'MacOS' || contains( inputs.os, 'windows-' ) && 'Windows' || 'Linux' }}
6060
permissions:
6161
contents: read
6262
runs-on: ${{ inputs.os }}

.github/workflows/reusable-test-gutenberg-build-process.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ jobs:
4141
# - Builds Gutenberg.
4242
# - Ensures version-controlled files are not modified or deleted.
4343
build-process-tests:
44-
name: Gutenberg running from ${{ inputs.directory }} / ${{ contains( inputs.os, 'macos-' ) && 'MacOS' || contains( inputs.os, 'windows-' ) && 'Windows' || 'Linux' }}
44+
name: ${{ contains( inputs.os, 'macos-' ) && 'MacOS' || contains( inputs.os, 'windows-' ) && 'Windows' || 'Linux' }}
4545
permissions:
4646
contents: read
4747
runs-on: ${{ inputs.os }}

0 commit comments

Comments
 (0)