Skip to content

fix: replace raw echo with standard output helpers in git/switchtomain#664

Merged
credfeto merged 1 commit into
mainfrom
fix/649-switchtomain-echo-helpers
Jun 5, 2026
Merged

fix: replace raw echo with standard output helpers in git/switchtomain#664
credfeto merged 1 commit into
mainfrom
fix/649-switchtomain-echo-helpers

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Jun 5, 2026

Summary

  • Added die, success, and info helper functions to git/switchtomain
  • Replaced all raw echo calls with the appropriate output helper
  • Fixed [ ! -z ... ][ -n ... ] for shellcheck/checkbashisms compliance
  • Behaviour is preserved: the "no default branch" case still continues to the next repository (uses info not die)

Closes #649

Test plan

  • shellcheck git/switchtomain passes
  • checkbashisms git/switchtomain passes
  • Script output now uses coloured / / indicators instead of plain echo

Prompt: Replace raw echo with output helpers in `git/switchtomain`

Closes #649
@github-actions github-actions Bot added the auto-pr Pull request created automatically label Jun 5, 2026
@dnyw4l3n13 dnyw4l3n13 marked this pull request as draft June 5, 2026 20:54
@dnyw4l3n13 dnyw4l3n13 added AI-Work Work for an AI Agent Urgent Urgent Priority labels Jun 5, 2026
@dnyw4l3n13 dnyw4l3n13 marked this pull request as ready for review June 5, 2026 21:07
@dnyw4l3n13
Copy link
Copy Markdown
Collaborator

All CI checks passed. PR is now ready for review.

Summary of changes:

  • Added die, success, and info output helper functions to git/switchtomain
  • Replaced all raw echo calls with the appropriate helpers
  • Fixed [ ! -z ... ][ -n ... ] for shellcheck/checkbashisms compliance
  • Behaviour preserved: "no default branch" case uses info (not die) so the loop continues to the next repository

@credfeto credfeto enabled auto-merge June 5, 2026 21:08
@credfeto
Copy link
Copy Markdown
Owner

credfeto commented Jun 5, 2026

Super-linter summary

Language Validation result
BASH Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

@credfeto credfeto merged commit f1db686 into main Jun 5, 2026
27 checks passed
@credfeto credfeto deleted the fix/649-switchtomain-echo-helpers branch June 5, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-Work Work for an AI Agent auto-pr Pull request created automatically Urgent Urgent Priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace raw echo with output helpers in git/switchtomain

2 participants