Skip to content

Add workflow to compare PR with the build server.#11279

Open
desrosj wants to merge 6 commits intoWordPress:trunkfrom
desrosj:add/build-script-results-check
Open

Add workflow to compare PR with the build server.#11279
desrosj wants to merge 6 commits intoWordPress:trunkfrom
desrosj:add/build-script-results-check

Conversation

@desrosj
Copy link
Member

@desrosj desrosj commented Mar 18, 2026

This changeset adds a new CI job to the "Check Built Files (PRs)" workflow that runs npm run build on the pull request's code and compares the resulting build directory against the corresponding branch of WordPress/WordPress.

The results are then posted as a pull request comment so that committers can review any differences without needing to check out the branch locally.

Trac ticket: Core-64878

Use of AI Tools

Claude was used to create the first draft of this PR.


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.

@desrosj desrosj requested a review from johnbillion March 18, 2026 03:28
@desrosj desrosj self-assigned this Mar 18, 2026
@github-actions
Copy link

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 props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props desrosj.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@desrosj desrosj force-pushed the add/build-script-results-check branch from 447c685 to 82d2d64 Compare March 18, 2026 17:53
@desrosj
Copy link
Member Author

desrosj commented Mar 18, 2026

I think this is ready to try. I've tested it in desrosj#505 while updating npm devDependencies and it seems to work well! The unexpected files in that list are due to how the build server does not clean up after itself every time and built files persist unless explicitly removed.

@desrosj desrosj requested a review from aaronjorbin March 18, 2026 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant