Skip to content

ci: wrap workflow run blocks in log groups#843

Merged
yungyuc merged 2 commits into
solvcon:masterfrom
rockleona:fix/issue-782
Jun 1, 2026
Merged

ci: wrap workflow run blocks in log groups#843
yungyuc merged 2 commits into
solvcon:masterfrom
rockleona:fix/issue-782

Conversation

@rockleona
Copy link
Copy Markdown
Collaborator

@rockleona rockleona commented May 30, 2026

Related to #782

Summary

Wrap the remaining multi-line GitHub Actions run: blocks with
::group:: / ::endgroup:: so long CI logs are folded by default.

Changes

  • Add log grouping to the remaining workflow steps in:
    • .github/workflows/devbuild.yml
    • .github/workflows/lint.yml
    • .github/workflows/nouse_install.yml
    • .github/workflows/profiling.yml
    • .github/workflows/send_email_on_fail.yml
  • Keep the workflow behavior unchanged; only improve log readability

Related to solvcon#782

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member

@yungyuc yungyuc left a comment

Choose a reason for hiding this comment

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

@rockleona Did you add the groups for everything, or have some contents missing? It's OK to include partial contents. Please leave inline annotations in the PR diff for what you include and what you do not include.

@yungyuc yungyuc added the build Build system and automation label May 30, 2026
@rockleona rockleona marked this pull request as draft May 30, 2026 08:06
ci: adding group in rest of the run stage
@rockleona rockleona marked this pull request as ready for review May 30, 2026 08:13
Copy link
Copy Markdown
Collaborator Author

@rockleona rockleona left a comment

Choose a reason for hiding this comment

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

I've scan all .yml files, all stages with run will have ::group:: annotation.

Comment on lines 55 to 56
run: |
echo "github.event_name: ${{ github.event_name }}"
Copy link
Copy Markdown
Collaborator Author

@rockleona rockleona May 30, 2026

Choose a reason for hiding this comment

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

But if there is only a single echo line, I won't place ::group:: annotation. Since I think it is annoying if there's only a single line after open the collapse.

Copy link
Copy Markdown
Member

@yungyuc yungyuc left a comment

Choose a reason for hiding this comment

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

LGTM

@ExplorerRay @chestercheng could you please help take a look?

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 792b969 into solvcon:master Jun 1, 2026
15 of 17 checks passed
@yungyuc yungyuc linked an issue Jun 1, 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.

Wrap more CI steps in a workflow log groups

3 participants