Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: improve the readability of oli command line error output #2016

Merged
merged 1 commit into from
Apr 17, 2023

Conversation

nayihz
Copy link
Contributor

@nayihz nayihz commented Apr 17, 2023

This PR just polish the output of oli command line when no subcommand was input.
Before:

# ./oli
Error: not handled

After:

# ./oli
OpenDAL Command Line Interface

Usage: oli [OPTIONS] <COMMAND>

Commands:
  cat   display object content
  cp    copy
  ls    ls
  rm    remove object
  stat  show object metadata
  help  Print this message or the help of the given subcommand(s)

Options:
      --config <config>  Path to the config file [default: /root/.config/oli/config.toml]
  -h, --help             Print help
  -V, --version          Print version

Signed-off-by: czybjtu <smartczy@outlook.com>
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo Xuanwo merged commit 3048310 into apache:main Apr 17, 2023
@nayihz nayihz deleted the feat_oli_cli branch April 17, 2023 06:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants