Skip to content
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

fix: CLI arguments not working #935

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

zdiff
Copy link
Contributor

@zdiff zdiff commented Nov 11, 2024

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Hotfix
  • Security patch
  • UI/UX improvement

Description

Fixes linutil --help command

Testing

Before PR

./target/release/linutil --help
error: unexpected argument found

After PR

 ./target/release/linutil --help
Usage: 

Options:
  -c, --config <CONFIG>      Path to the configuration file
  -t, --theme <THEME>        Set the theme to use in the application [default: default] [possible values: default, compatible]
  -y, --skip-confirmation    Skip confirmation prompt before executing commands
      --override-validation  Show all available options, disregarding compatibility checks (UNSAFE)
      --size-bypass          Bypass the terminal size limit
  -h, --help                 Print help

Issues / other PRs related

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

@adamperkowski adamperkowski changed the title Fixes #934 fix: CLI arguments not working Nov 11, 2024
@adamperkowski adamperkowski added the bug Something isn't working label Nov 11, 2024
@lj3954 lj3954 mentioned this pull request Nov 11, 2024
7 tasks
@jeevithakannan2
Copy link
Contributor

jeevithakannan2 commented Nov 11, 2024

On second thought we can keep the default-features. Default features is actually good with colors and suggestions, no reason to remove them

@adamperkowski adamperkowski added the rust Pull requests that update Rust code label Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CLI arguments not working
4 participants