fix: replace raw echo with standard output helpers in git/update-repos-personal#661
Conversation
…s-personal Replace the non-standard die() function and raw echo calls with the project-standard die/info/success helpers using coloured printf output. Also fix shebang spacing (#! /bin/sh → #!/bin/sh). Closes #652 Prompt: Work on issue #652 in credfeto/scripts - replace raw echo with output helpers (die, info, success) in git/update-repos-personal
StatusChanges are committed and pushed. PR is in draft. CI is running — |
success() was added but is never called in this script; the script delegates to fetch which handles its own success output per repo.
Status: Ready for ReviewAll CI checks passed. Code review complete. PR marked ready. Changes
|
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
Status: CompleteAll CI checks passing. PR body updated to accurately reflect the final changes (the No outstanding review comments or CHANGES_REQUESTED reviews. PR is ready for merge. |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
2 participants
Summary
#! /bin/sh→#!/bin/sh)die()function with the project-standard implementation using colouredprintfoutput directed to stderrinfo()helper function and replace rawecho "Script Dir: $BASEDIR"withinfo "Script Dir: $BASEDIR"Closes #652
Test plan