Opened 2 months ago
Last modified 8 weeks ago
#64878 new feature request
Create a workflow that compares a PR with the build server
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | 7.1 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Build/Test Tools | Keywords: | has-patch |
| Focuses: | Cc: |
Description
When a pull request modifies files that affect the build output, it is currently difficult to know whether the resulting build directory would differ from what the build server currently has committed.
It would be helpful to have a workflow that compares the contents of the build directory within a pull request with WordPress/WordPress to make sure that there are no unexpected changes.
Change History (3)
This ticket was mentioned in PR #11279 on WordPress/wordpress-develop by @desrosj.
2 months ago
#1
@desrosj commented on PR #11279:
2 months ago
#2
I think this is ready to try. I've tested it in https://github.com/desrosj/wordpress-develop/pull/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.
This changeset adds a new CI job to the "Check Built Files (PRs)" workflow that runs
npm run buildon the pull request's code and compares the resultingbuilddirectory 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.