Skip to content

fix: replace raw echo with output helpers in git/init-preview#670

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
fix/644-replace-echo-with-helpers-in-init-preview
Open

fix: replace raw echo with output helpers in git/init-preview#670
github-actions[bot] wants to merge 1 commit into
mainfrom
fix/644-replace-echo-with-helpers-in-init-preview

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

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

Summary

  • Replaced the non-standard local die() implementation (using bare echo) with the canonical printf-based version that writes to stderr
  • Added standard success() and info() output helpers
  • Replaced bare echo "Using Branch: $BRANCH" with info "Using Branch: $BRANCH"
  • Added success confirmation at end of successful execution chain

Closes #644

Test plan

  • shellcheck git/init-preview passes
  • checkbashisms git/init-preview passes
  • Script behaviour is preserved — only output formatting changed

Closes #644

Prompt: Work on issue #644 in credfeto/scripts. Replace raw echo with output helpers (die/info/success) in git/init-preview.
@github-actions github-actions Bot added the auto-pr Pull request created automatically label Jun 6, 2026
@dnyw4l3n13 dnyw4l3n13 marked this pull request as draft June 6, 2026 13:04
@dnyw4l3n13 dnyw4l3n13 added AI-Work Work for an AI Agent Urgent Urgent Priority labels Jun 6, 2026
@dnyw4l3n13 dnyw4l3n13 marked this pull request as ready for review June 6, 2026 14:17
@dnyw4l3n13
Copy link
Copy Markdown
Collaborator

All CI checks have passed. No review comments or outstanding work. Marking as ready for review.

  • build-pre-release
  • pull-request
  • Apply Template and Generate Description
  • Update Change Type Checkboxes

@credfeto
Copy link
Copy Markdown
Owner

credfeto commented Jun 6, 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

@dnyw4l3n13
Copy link
Copy Markdown
Collaborator

All CI checks pass and there are no outstanding review comments. PR is ready for merge.

CI summary:

  • build-pre-release
  • lint-code
  • change-log-entry-is-in-unreleased
  • include-changelog-entry
  • valid-changelog
  • does-not-contain-secrets
  • dependency-review
  • has-no-merge-conflicts
  • has-no-file-or-folder-case-sensitivity-issues
  • no-ignored-files
  • no-merge-commits
  • pull-request
  • BASH super-linter ✅

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/init-preview

2 participants