Skip to content

Fix input syntax in CLI help example - #140

Open
brbousnguar wants to merge 1 commit into
mulesoft:masterfrom
brbousnguar:fix/issue-113
Open

Fix input syntax in CLI help example#140
brbousnguar wants to merge 1 commit into
mulesoft:masterfrom
brbousnguar:fix/issue-113

Conversation

@brbousnguar

Copy link
Copy Markdown

What Problem This Solves

The input example shown by dw --help and reproduced in the README uses a space between the input name and path. Picocli parses the -i option as a map, so its required syntax is name=path; copying the existing example results in a command that does not run.

Why This Change Was Made

  • Correct the help footer to use -i payload=<fullPathToUser.json>.
  • Keep the README's captured help output consistent with the CLI.
  • Add a focused test that executes the CLI with the documented name=path input form.

User Impact

Users can copy the input-file example from dw --help or the README and run it successfully.

Evidence

  • ./gradlew native-cli:test
  • 16 tests passed with zero failures (15 DataWeaveCLITest tests and 1 DependencyManagerTest)
  • git diff --check

Fixes #113

@brbousnguar
brbousnguar requested a review from a team as a code owner July 27, 2026 12:24

@mlischetti mlischetti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mlischetti

Copy link
Copy Markdown
Contributor

@brbousnguar thanks for your contribution! In order to proceed with the merge, the commits must have verified signatures. Can you please verify it. Thanks!

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.

Update needed for output from --help option

2 participants