Opened 7 months ago
Last modified 3 months ago
#64878 new feature request
Create a workflow that compares a PR with the build server
| Reported by: | desrosj | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Future Release |
| Component: | Build/Test Tools | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
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 (7)
This ticket was mentioned in PR #11279 on WordPress/wordpress-develop by @desrosj.
7 months ago
#1
@desrosj commented on PR #11279:
7 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.
#3
@
6 months ago
- Milestone 7.0 → 7.1
While this would have been helpful to debug some of the unexpected changes to the built package during this cycle, I did not have time to wrap this up. It's not critical for 7.0, so punting.
This ticket was mentioned in Slack in #core by adrianduffell. View the logs.
3 months ago
#5
@
3 months ago
With the 7.1 beta approaching, this has a chance to be included if the PR can be reviewed in the coming days.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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.