github: workflows: remove zmain build test#10662
github: workflows: remove zmain build test#10662lgirdwood wants to merge 1 commit intothesofproject:mainfrom
Conversation
This just adds more noise than signal to the CI results. Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com>
lgirdwood
changed the title
There was a problem hiding this comment.
Pull request overview
This PR removes the zmain Zephyr build variant from the Zephyr GitHub Actions workflow to reduce CI noise.
Changes:
- Removed
zmainfrom thezephyr_revisionmatrix in the Linux build job. - Deleted the
zmain-specific manifest override generation andwest updatelogic, leaving only themnfstpath.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -113,7 +113,6 @@ jobs: | |||
| # Search "zephyr_revision" and see below what they expand to. | |||
| zephyr_revision: [ | |||
| mnfst, # special value: don't override sof/west.yml | |||
There was a problem hiding this comment.
In the zephyr_revision flow sequence, mnfst still has a trailing comma after removing zmain. YAML flow sequences are typically strict about trailing commas, and this can cause the workflow to fail to parse. Remove the trailing comma (or switch to block-style list) now that there's only one entry.
| mnfst, # special value: don't override sof/west.yml | |
| mnfst # special value: don't override sof/west.yml |
| echo 'Unknown matrix.zephyr_revision'; exit 1 | ||
| fi |
There was a problem hiding this comment.
After removing the zmain option, there is still a comment later in this job that says "especially in the "zmain" case". Please update/remove that stale reference so the workflow documentation matches the supported zephyr_revision values.
This just adds more noise than signal to the CI results.