google / google/google-java-format

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

Offen
#1,094 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
6.2k
Forks
937
Ø Merge
6 Min.
Gemergte PRs (30 T.)
3

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.