Move Cortex-M backend tests from trunk.yml to pull.yml#18308
Move Cortex-M backend tests from trunk.yml to pull.yml#18308psiddh merged 2 commits intopytorch:mainfrom
Conversation
The test-mcu-cortex-m-backend CI job runs all Cortex-M pytests but was only in trunk.yml, meaning zero tests ran at diff time. Move it to pull.yml so regressions are caught on-diff before merge. Authored with Claude.
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/18308
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 Cancelled Job, 3 Unrelated FailuresAs of commit 9dff79b with merge base 569cf41 ( CANCELLED JOB - The following job was cancelled. Please retry:
BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
There was a problem hiding this comment.
Pull request overview
Moves the Cortex-M backend CI job (test-mcu-cortex-m-backend) from the trunk workflow into the pull workflow so Cortex-M pytest regressions are caught during PR validation rather than only after merge.
Changes:
- Removed
test-mcu-cortex-m-backendjob from.github/workflows/trunk.yml. - Added the same
test-mcu-cortex-m-backendjob to.github/workflows/pull.yml(builds the Cortex-M test runner and runsbackends/cortex_m/testvia pytest).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/trunk.yml | Removes the Cortex-M backend test job from the trunk workflow job set. |
| .github/workflows/pull.yml | Adds the Cortex-M backend test job to PR CI so it runs pre-merge. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The test-mcu-cortex-m-backend CI job runs all Cortex-M pytests but was only in trunk.yml, meaning zero tests ran at diff time. Move it to pull.yml so regressions are caught on-diff before merge.
Authored with Claude.