#52983 closed defect (bug) (fixed)
Prevent PHPUnit tests from running on `push` for forks and private mirrors
Reported by: | desrosj | Owned by: | desrosj |
---|---|---|---|
Milestone: | 5.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Build/Test Tools | Keywords: | has-patch fixed-major |
Focuses: | Cc: |
Description
In [50441], the PHPUnit workflow was updated to allow WordPress to be run from src
. The workflow was reorganized to take advantage of this change, but it has resulted in unnecessary workflow runs on forks and private mirrors.
Before [50441], the first job (which built and prepared WordPress for testing) was required to complete before the second one (that performed all the actual testing) could take place. Because the first job now only cancels incomplete workflow runs in pull requests, the testing job runs always.
This can be fixed with an if:
statement to ensure the jobs only run in the correct context.
Attachments (1)
Change History (9)
Note: See
TracTickets for help on using
tickets.
This will need to be backported back to the 5.2 branch.