Skip to content

CI: wrap every setup composite step in a workflow log group#781

Merged
yungyuc merged 1 commit into
solvcon:masterfrom
yungyuc:ci/reporting
May 14, 2026
Merged

CI: wrap every setup composite step in a workflow log group#781
yungyuc merged 1 commit into
solvcon:masterfrom
yungyuc:ci/reporting

Conversation

@yungyuc
Copy link
Copy Markdown
Member

@yungyuc yungyuc commented May 14, 2026

Add the pairs of ::group:: and ::endgroup:: markers around the run-block body of every step in setup_linux and setup_macos composite actions. Reviewers can fold the output of each step and navigate quickly without scrolling more than 1,000 lines.

Related to #778

Add the pairs of `::group::` and `::endgroup::` markers around the run-block
body of every step in `setup_linux` and `setup_macos` composite actions.
Reviewers can fold the output of each step and navigate quickly without
scrolling more than 1,000 lines.

Related to solvcon#778
@yungyuc yungyuc requested a review from ExplorerRay May 14, 2026 14:42
@yungyuc yungyuc self-assigned this May 14, 2026
@yungyuc yungyuc added the build Build system and automation label May 14, 2026
@yungyuc yungyuc removed the request for review from ExplorerRay May 14, 2026 14:43
@ExplorerRay
Copy link
Copy Markdown
Collaborator

This looks great, is this a builtin feature in Github?

@yungyuc yungyuc marked this pull request as ready for review May 14, 2026 15:06
@yungyuc
Copy link
Copy Markdown
Member Author

yungyuc commented May 14, 2026

This looks great, is this a builtin feature in Github?

It is. The kind Claude taught me.

@ExplorerRay
Copy link
Copy Markdown
Collaborator

This looks great, is this a builtin feature in Github?

It is. The kind Claude taught me.

OK, I found the reference for this.
https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-commands#grouping-log-lines

Copy link
Copy Markdown
Collaborator

@ExplorerRay ExplorerRay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@yungyuc yungyuc merged commit f8c945c into solvcon:master May 14, 2026
16 of 17 checks passed
@yungyuc yungyuc deleted the ci/reporting branch May 14, 2026 22:45
@yungyuc yungyuc linked an issue May 14, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Build system and automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI hides the toolchain reporting too deeply in the log

2 participants