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

Misleading message for --dry-run with --replace #1094

Open
norrisgc opened this issue Apr 7, 2024 · 0 comments
Open

Misleading message for --dry-run with --replace #1094

norrisgc opened this issue Apr 7, 2024 · 0 comments

Comments

@norrisgc
Copy link

norrisgc commented Apr 7, 2024

If you specify both --dry-run (or -n) and --replace (or -replace, -r or -i) you get a message stating "cannot use --dry-run and --in-place at the same time".

The problem is that there is no --in-place option. inPlace is however the internal variable name used for --replace. There's two possible fixes:

  1. Change the message. This could affect anything which tries to use the message in some kind of script.
  2. Add the --in-place option at line 68 in CommandLineOptionsParser.java and related help text. This should not affect anything else.
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

No branches or pull requests

1 participant