Skip to content

Conversation

@szczys
Copy link
Contributor

@szczys szczys commented Jan 6, 2026

Only post alls-green console summaries if the were matrix tests that did not succeed.

Updates to forked alls-green action:

re-actors/alls-green@unstable/v1...golioth:alls-green:szczys/configure-action-summary-output

A PR to the upstream repo is open here: re-actors/alls-green#36

Behavior of successful verify steps

Successful builds do not generate a summary card, but the output of alls-green is still shown in the verify job output:
image

Behavior when all matrix steps are not successful

Cards are shown in the summary when the verify job is not successful:
image

resolves: https://github.com/golioth/firmware-issue-tracker/issues/900

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Visit the preview URL for this PR (updated for commit 3ae93b0):

https://golioth-firmware-sdk-doxygen-dev--pr896-szczys-quiet-a-0wqngeun.web.app

(expires Tue, 13 Jan 2026 22:10:25 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

Base automatically changed from szczys/twister-device-parallel to main January 6, 2026 22:08
Only post alls-green console summaries if the were matrix tests that did
not succeed

Signed-off-by: Mike Szczys <[email protected]>
@szczys szczys force-pushed the szczys/quiet-alls-green branch from d545e77 to 3ae93b0 Compare January 6, 2026 22:09
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Code Coverage

Code Coverage

Package Line Rate Branch Rate Health
include.golioth 75% 50%
port.linux 75% 42%
port.utils 58% 46%
port.zephyr 79% 28%
src 74% 29%
Summary 75% (2635 / 3536) 29% (1446 / 4940)

@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 4 files with indirect coverage changes

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.

2 participants