Skip to content

Fix: update --add-dir help text to use plural DIRS placeholder#232

Open
watcharaponthod-code wants to merge 1 commit into
CortexLM:mainfrom
watcharaponthod-code:main
Open

Fix: update --add-dir help text to use plural DIRS placeholder#232
watcharaponthod-code wants to merge 1 commit into
CortexLM:mainfrom
watcharaponthod-code:main

Conversation

@watcharaponthod-code
Copy link
Copy Markdown

This PR addresses the inconsistency identified in PlatformNetwork/bounty-challenge#53507.

Changes

  • Updated the value_name of the --add-dir argument from DIR to DIRS in src/cortex-cli/src/cli/args.rs.
  • This matches the plural context of the help description ("Additional directories") and clarifies that multiple directories can be provided (via repeated flags), following standard CLI conventions.

Verification

  • Verified the source code change in args.rs.
  • Ensures that cortex --help will now display --add-dir <DIRS> instead of the confusing singular form.

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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