Skip to content

fix(ci): Skip build on empty build matrix.#3271

Merged
caksoylar merged 1 commit into
zmkfirmware:mainfrom
aadcg:issue-3179
Mar 16, 2026
Merged

fix(ci): Skip build on empty build matrix.#3271
caksoylar merged 1 commit into
zmkfirmware:mainfrom
aadcg:issue-3179

Conversation

@aadcg

@aadcg aadcg commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

Closes #3179.


I tested the CI here.

It skips the build and the CI routine returns as successful. I'm not sure how to retrieve the log message from the skipped job, but I can look into it.

CC @joelspadin.

@aadcg
aadcg requested a review from a team as a code owner March 3, 2026 21:23
@caksoylar

Copy link
Copy Markdown
Contributor

Maybe a notice message annotation would be better so the message is surfaced to the main build page? https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-commands#setting-a-notice-message

Similar to how ::warning is used in https://github.com/zmkfirmware/zmk/blob/main/.github/workflows/build-user-config.yml.

@aadcg

aadcg commented Mar 16, 2026

Copy link
Copy Markdown
Contributor Author

Maybe a notice message annotation would be better so the message is surfaced to the main build page?

@caksoylar, sounds good. I've pushed an updated commit with that change.

To check how the notice message looks like, please see https://github.com/aadcg/test-zmk-config/actions/runs/23148617644.

Sorry for the late reply.

@caksoylar
caksoylar merged commit df2e979 into zmkfirmware:main Mar 16, 2026
5 checks passed
@caksoylar

Copy link
Copy Markdown
Contributor

Thanks for the fix!

phuertay added a commit to phuertay/zmk that referenced this pull request Mar 18, 2026
fix(ci): Skip build on empty build matrix (zmkfirmware#3271)
@aadcg
aadcg deleted the issue-3179 branch March 19, 2026 20:13
badjeff pushed a commit to badjeff/zmk that referenced this pull request Mar 28, 2026
dgct pushed a commit to dgct/zmk that referenced this pull request Apr 28, 2026
caksoylar pushed a commit to caksoylar/zmk that referenced this pull request May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

User config build workflow fails when build matrix is empty

2 participants