Try: Build WP once for PHPUnit test suite. - #12779
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
| env: | ||
| PUPPETEER_SKIP_DOWNLOAD: true | ||
|
|
||
| - name: Build WordPress |
There was a problem hiding this comment.
This reusable workflow also serves test-coverage.yml, whose two coverage jobs do not pass wordpress-build-artifact
In this PR’s Code Coverage run, this step built WordPress in prepare-gutenberg, then both coverage jobs received an empty input and ran their fallback builds too. That adds a serial build and artifact upload without replacing either child build.
Could we pass wordpress-build from test-coverage.yml, or make this producer work opt-in?
AI use: prepared with gpt-5.6-sol
Runs
npm run build:devin the newly createdprepare-gutenbergreusable workflow for the PHPUnit tests.This allows the step to be bypassed in each test, saving about 30 seconds of runtime.
As the local environment uses NPM for the docker pull and configuration, the
npm cistep is still required. That can be a project for another time.Trac ticket: https://core.trac.wordpress.org/ticket/65770
Use of AI Tools
AI assistance: Yes
Tool(s): GitHub Copilot
Used for: Initial drafting of this in another PR that also included some docker changes. The creation of this PR built upon portions of that but is much reduced in scope. AI was not used for the generation of this PR per se.
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.