Skip to content

app: display help message when command is not specified #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 22, 2025

Conversation

psurply
Copy link
Contributor

@psurply psurply commented May 21, 2025

The CLI command is now mandatory. This allows the help message to be displayed when no command is provided.

$ iclg
Extract and decode Intel Crash Log records.

Usage: iclg [OPTIONS] <COMMAND>
...

This change also refactors the main function to prevent the CLI from being parsed twice.

The CLI command is now mandatory. This allows the help message to
be displayed when no command is provided.

    $ iclg
    Extract and decode Intel Crash Log records.

    Usage: iclg [OPTIONS] <COMMAND>
    ...

This change also refactors the main function to prevent the CLI from
being parsed twice.

Signed-off-by: Surply, Pierre <pierre.surply@intel.com>
@psurply psurply force-pushed the dev/app_command branch from db5463f to 23a3d65 Compare May 21, 2025 07:56
@psurply psurply requested a review from jlrivasp May 21, 2025 07:58
Copy link
Contributor

@jlrivasp jlrivasp left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the PR, it looks good to me

@jlrivasp jlrivasp merged commit 4c7b6b3 into intel:main May 22, 2025
5 checks passed
@psurply psurply deleted the dev/app_command branch June 12, 2025 21:31
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.

2 participants