Skip to content

add action-summary-output option#36

Open
szczys wants to merge 1 commit intore-actors:unstable/v1from
golioth:szczys/configure-action-summary-output
Open

add action-summary-output option#36
szczys wants to merge 1 commit intore-actors:unstable/v1from
golioth:szczys/configure-action-summary-output

Conversation

@szczys
Copy link

@szczys szczys commented Jan 6, 2026

add functionality to control when a card is added to the Github action summary page.
Valid options:

action-summary-output behavior
all results shown in both stderr and action status.
none results shown only in stderr.
quiet-on-success matrix success only shown only in stderr, all other output follows same behavior as 'all'.

In all cases, the output of alls-green is still written to stderr. The special case quiet-on-success, any alls-green job in which all matrix steps pass will not publish a card to the action summary but in all other cases (non-success of matrix) the card will still be published.

Resolves #35

add functionality to control when a card is added to the Github action
summary page. Valid options: all, none, quiet-on-success

Signed-off-by: Mike Szczys <mike@golioth.io>
@webknjaz
Copy link
Member

webknjaz commented Jan 7, 2026

There's conflicting changes of higher priority that I haven't fished/pushed yet. I'll have to get back to that first, meaning you should expect having to rebase interactively at some point.

But meanwhile, could you document the use case? It sounds like you might be integrating the action incorrectly.

@szczys
Copy link
Author

szczys commented Jan 7, 2026

There's conflicting changes of higher priority that I haven't fished/pushed yet. I'll have to get back to that first, meaning you should expect having to rebase interactively at some point.

But meanwhile, could you document the use case? It sounds like you might be integrating the action incorrectly.

I'm happy to rebase as necessary. I've replied to #35 with my use-case. Please let me know if you'd prefer to continue the discussion on this PR instead.

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.

Summary cards are not configurable

2 participants

Comments