Skip to content

fix: list all CLI options - #359

Open
niksy wants to merge 1 commit into
sarbbottam:masterfrom
niksy:list-cli-options
Open

fix: list all CLI options#359
niksy wants to merge 1 commit into
sarbbottam:masterfrom
niksy:list-cli-options

Conversation

@niksy

@niksy niksy commented Feb 16, 2025

Copy link
Copy Markdown

CLI currently only lists some options. This PR adds support for listing all available options.

Before

Options:
      --help     Show help                                             [boolean]
      --version  Show version number                                   [boolean]
  -i, --include                                 [string] [choices: "deprecated"]
      --ext                                           [array] [default: [".js"]]

After

Options:
      --help                                Show help                  [boolean]
      --version                             Show version number        [boolean]
  -i, --include                                 [string] [choices: "deprecated"]
      --ext                                           [array] [default: [".js"]]
  -c, --getCurrentRules, --current                    [boolean] [default: false]
  -p, --getPluginRules, --plugin                      [boolean] [default: false]
  -a, --getAllAvailableRules,
  --all-available                                     [boolean] [default: false]
  -u, --getUnusedRules, --unused                      [boolean] [default: false]
  -d, --getDeprecatedRules, --deprecated              [boolean] [default: false]
  -n                                                  [boolean] [default: false]
      --error                                          [boolean] [default: true]
      --core                                           [boolean] [default: true]
      --flatConfig                                    [boolean] [default: false]

Question: are these options really callable?

  • --getCurrentRules
  • --getPluginRules
  • --getAllAvailableRules
  • --getUnusedRules
  • --getDeprecatedRules

It seems like this could be wrong option interpretation, but current tests and yargs arguments parsing calls for these options.

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