Skip to content

fix: print help to stdout and support -h alias - #16

Merged
alexanderbsingh merged 7 commits into
mainfrom
fix-help-behavior
Aug 6, 2026
Merged

fix: print help to stdout and support -h alias#16
alexanderbsingh merged 7 commits into
mainfrom
fix-help-behavior

Conversation

@alexanderbsingh

@alexanderbsingh alexanderbsingh commented Jul 22, 2026

Copy link
Copy Markdown
Contributor
  • Add -h as an alias for --help.
  • Print help output to stdout and usage errors to stderr.
  • Update README option documentation.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the bifrost CLI help/usage behavior to better align with common CLI conventions by adding a -h alias, routing help text to stdout, and routing usage/errors to stderr, with added tests and README updates.

Changes:

  • Add -h as an alias for --help and update README option documentation.
  • Route help output to stdout, while keeping usage/errors on stderr.
  • Add CLI tests that assert correct streams and exit codes for help/no-arg scenarios.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
README.md Documents -h alongside --help in the options table.
internal/bifrost/cli.go Adds -h alias and refactors help/usage printing to support stdout/stderr routing and “help after command” handling.
internal/bifrost/cli_test.go Adds tests and a helper to capture both stdout and stderr for help/no-arg behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread internal/bifrost/cli.go Outdated

This comment was marked as resolved.

This comment was marked as resolved.

@alexanderbsingh
alexanderbsingh marked this pull request as ready for review July 23, 2026 09:30
@alexanderbsingh
alexanderbsingh requested a review from konrader July 23, 2026 15:10
@alexanderbsingh
alexanderbsingh merged commit 5d97f11 into main Aug 6, 2026
2 checks passed
@alexanderbsingh
alexanderbsingh deleted the fix-help-behavior branch August 6, 2026 11:11
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.

3 participants