Quality polish: DRY, bug fixes, step summaries, dep bumps#182
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR improves the maintainability and observability of the wishabi/github-actions action suite by extracting shared branch/SHA logic into a reusable composite action, adding CI step summaries, and updating dependencies/runtimes.
Changes:
- Extract duplicated branch + short-SHA extraction into
common/branch-infoand reuse across composite actions. - Add
$GITHUB_STEP_SUMMARYmarkdown summaries to multiple Go actions for quicker CI signal. - Bump several GitHub Actions dependencies (Node 16→20,
setup-node@v3→v4, Slack action version bump) and remove the deprecatedsonarqube-scanaction.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
sonarqube-scan/action.yml |
Removes the deprecated SonarQube composite action. |
common/branch-info/action.yml |
Adds reusable composite action for branch name + short SHA extraction. |
slack/action.yml |
Uses shared branch-info and updates Slack action version + modern $GITHUB_OUTPUT output. |
go/test/test.sh |
Hardens script invocation and adjusts how test output is captured. |
go/test/action.yml |
Adds step summary + configurable artifact retention. |
go/build/action.yml |
Uses branch-info, adds build summary, bumps Slack action version. |
go/lint/action.yml |
Uses branch-info, logs tool versions, adds lint summary, bumps Slack action version. |
go/build-and-test/action.yml |
Uses branch-info and bumps Slack action version. |
go/smoke-test/action.yml |
Uses branch-info, bumps DataDog + Slack action versions, adds smoke test summary. |
go/deps/action.yml |
Adds tool-version logging for easier debugging. |
deploy/eks/action.yml |
Uses branch-info, bumps Slack action version, fixes “Deplloyment” typo. |
ruby/test/action.yml |
Adds configurable artifact retention days. |
ruby/lint/action.yml |
Adds configurable artifact retention days. |
action.yml |
Bumps JavaScript action runtime to node20. |
.github/workflows/test.yml |
Updates Node to 20 and setup-node to v4; adds NODE_OPTIONS workaround. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
chriskarlin
approved these changes
Mar 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
common/branch-infocomposite action, reuse across all actionsset -eu, fix incorrect variable references, safer parameter defaults$GITHUB_STEP_SUMMARYmarkdown tables to Go build, test, lint, smoke-test, and deploy actions for at-a-glance CI resultsactions/setup-node@v3→v4,node-version: 16→20;slackapi/slack-github-action@v1.24.0→v1.27.1sonarqube-scanactionNo breaking changes — all new inputs are optional with backward-compatible defaults.
Validated on consuming repos
Files changed
common/branch-info/action.yml— shared branch/commit info stepgo/build,go/test,go/lint,go/build-and-test,go/deps,go/smoke-test,deploy/eks,slack,ruby/lint,ruby/test— DRY refactors + summaries.github/workflows/test.yml— Node.js version bumpsonarqube-scan/action.ymlTest plan