Skip to content

Organize bash libraries with per-module docs and tests#1

Merged
codeforester merged 1 commit intomainfrom
hpr/first-steps
Apr 11, 2026
Merged

Organize bash libraries with per-module docs and tests#1
codeforester merged 1 commit intomainfrom
hpr/first-steps

Conversation

@codeforester
Copy link
Copy Markdown
Owner

fix: correct several bugs in lib_std.sh

  • Fix color gate logic: color is "0"/"1" so -z check was always false, causing colors to always enable in interactive terminals regardless of the --color flag
  • Add COLOR_BOLD to the readonly declaration (was set but not protected)
  • Add missing >&2 redirect on unsupported OSTYPE error message
  • Fix wait_for_enter to actually wait for Enter by removing -n1
  • Fix assert_integer_range error message to include the tested value and use clearer range notation
  • Remove spurious "$@" argument passed to dump_trace (ignored but wrong)
  • Remove redundant >&2 on log_debug call (already writes to stderr)
  • Fix Usaage typo in safe_mkdir comment

fix: correct several bugs in lib_std.sh

- Fix color gate logic: __color__ is "0"/"1" so -z check was always
  false, causing colors to always enable in interactive terminals
  regardless of the --color flag
- Add COLOR_BOLD to the readonly declaration (was set but not protected)
- Add missing >&2 redirect on unsupported OSTYPE error message
- Fix wait_for_enter to actually wait for Enter by removing -n1
- Fix assert_integer_range error message to include the tested value
  and use clearer range notation
- Remove spurious "$@" argument passed to dump_trace (ignored but wrong)
- Remove redundant >&2 on log_debug call (already writes to stderr)
- Fix Usaage typo in safe_mkdir comment
@codeforester codeforester merged commit 6ea37a3 into main Apr 11, 2026
0 of 3 checks passed
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.

1 participant