[cconstable]

What was changed

• Added a Required PR Checks job to per-pr.yml. It runs after the jobs that are required to merge today and fails if any of them failed or was cancelled.
• Moved the heavy tests from heavy.yml into per-pr.yml as Heavy Tests, since a job can only wait on jobs in its own file.
• The macos-intel runs of unit and integration tests become required. They already run on every PR, and a job can only wait on a whole matrix job, not on individual entries.

Why?

Branch protection lists required checks by exact name, including one per OS for unit and integration tests. Any CI change also needs a branch protection change made at the same moment. With one roll-up job, those changes only touch this repo. Branch protection will be switched to require only Required PR Checks after this merges.

[cconstable]

needed to move this into this file so the required-pr-checks check can reference it in it's needs: block

[Sushisource]

The yaml ID for this job is a bit weird now though. Should change to heavy-tests or something

[chris-olszewski]

@Sushisource's comment still stands. A byproduct of this is I will finally just kill the macos-intel job since that was previously non-blocking and this makes it blocking.