Skip to content

BUILD-9431 Group GitHub Actions logs for readability#221

Merged
bwalsh434 merged 2 commits intomasterfrom
BUILD-9431-group-action-logs-readability
Feb 20, 2026
Merged

BUILD-9431 Group GitHub Actions logs for readability#221
bwalsh434 merged 2 commits intomasterfrom
BUILD-9431-group-action-logs-readability

Conversation

@bwalsh434
Copy link
Contributor

@bwalsh434 bwalsh434 commented Feb 18, 2026

Wrap verbose operations in ::group::/::endgroup:: across all build scripts so GitHub Actions logs are folded by default, showing a compact summary of build steps with expandable details.

Test Plan

Temporary PRs have been generated for the dummy repos that test these changes! See below:

Repo Actions tested PR
sonar-dummy-gradle-oss build-gradle, config-gradle, get-build-number, promote SonarSource/sonar-dummy-gradle-oss#332
sonar-dummy-maven-enterprise build-maven, config-maven, get-build-number, promote https://github.com/SonarSource/sonar-dummy-maven-enterprise/pull/120
sonar-dummy build-maven, get-build-number, promote https://github.com/SonarSource/sonar-dummy/pull/556
sonar-dummy-js build-npm, config-npm, promote SonarSource/sonar-dummy-js#119
sonar-dummy-python-oss build-poetry, promote SonarSource/sonar-dummy-python-oss#79
sonar-dummy-yarn build-yarn, promote https://github.com/SonarSource/sonar-dummy-yarn/pull/44

@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Feb 18, 2026

BUILD-9431

@bwalsh434 bwalsh434 marked this pull request as ready for review February 18, 2026 23:05
@bwalsh434 bwalsh434 requested a review from a team as a code owner February 18, 2026 23:05
Copilot AI review requested due to automatic review settings February 18, 2026 23:05
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds GitHub Actions log grouping across all build scripts to improve log readability by wrapping verbose operations in ::group::/::endgroup:: markers. These groups allow GitHub Actions to collapse sections by default, showing a compact summary of build steps with expandable details.

Changes:

  • Added log grouping to all build scripts (Maven, Gradle, NPM, Yarn, Poetry) and support scripts (promote, version setters, artifact deployment)
  • Updated all corresponding test specifications to verify the new grouped output format
  • Groups are consistently applied to tool checks, environment configuration, build steps, Sonar analysis, and deployment operations

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated no comments.

Show a summary per file
File Description
promote/promote.sh Added groups for tool checks, promotion configuration, artifact promotion, and GitHub notifications
config-maven/set_maven_project_version.sh Added groups for tool checks and version setting
config-gradle/set_gradle_project_version.sh Added groups for tool checks and version setting
build-yarn/build.sh Added groups for tool checks, environment setup, version setting, dependencies, tests, build, and deployment
build-poetry/build.sh Added groups for tool checks, environment setup, version setting, dependencies, build, Sonar analysis, and deployment
build-npm/build.sh Added groups for tool checks, git fetch, dependencies, tests, build, and deployment
build-maven/deploy-artifacts.sh Added groups for JFrog configuration and public/private artifact deployment
build-maven/build.sh Added groups for tool checks, git fetch, and Maven build
build-gradle/build.sh Added groups for tool checks, environment setup, and Gradle build
spec/*.sh (8 files) Updated test expectations to verify the new grouped output format with correct line numbers and assertions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

bwalsh434 and others added 2 commits February 19, 2026 22:39
Wrap verbose operations in ::group::/::endgroup:: across all build
scripts so GitHub Actions logs are folded by default, showing a compact
summary of build steps with expandable details.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@bwalsh434 bwalsh434 force-pushed the BUILD-9431-group-action-logs-readability branch from 1ab5a3c to e50fb05 Compare February 20, 2026 04:39
@sonarqubecloud
Copy link

SonarQube reviewer guide

Review in SonarQube

Summary: Adds GitHub Actions workflow grouping (::group:: and ::endgroup::) to build and deployment scripts across all package managers (Gradle, Maven, npm, Poetry, Yarn) and promotion workflow for improved log organization and readability.

Review Focus:

  • Verify that group markers are correctly paired and logically organized around logical workflow steps
  • Ensure test expectations accurately reflect the new output format with group markers
  • Confirm no functional changes to build logic, only output formatting additions
  • Check that comments in code (e.g., "orchestrate_sonar_platforms emits its own groups") correctly document which functions handle their own grouping

Start review at: build-maven/build.sh. This is the most complex script with the most test updates, making it the best indicator of whether the grouping strategy is sound and consistently applied across the codebase.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues
0 Dependency risks

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@bwalsh434 bwalsh434 merged commit 5b58ea1 into master Feb 20, 2026
11 checks passed
@bwalsh434 bwalsh434 deleted the BUILD-9431-group-action-logs-readability branch February 20, 2026 04:52
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.

3 participants