Skip to content

Commit e2cf9d2

Browse files
authored
Rephrase source code fix interactive messages for clarity (#390)
* rewrite source code fix user messages for clarity
1 parent 8cfbab6 commit e2cf9d2

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

src/main/java/org/apache/maven/plugins/javadoc/AbstractFixJavadocMojo.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -569,7 +569,7 @@ private boolean preCheck() throws MojoExecutionException {
569569
}
570570

571571
if (!settings.isInteractiveMode()) {
572-
getLog().error("Maven is not attempt to interact with the user for input. "
572+
getLog().error("Maven is not interacting with the user for input. "
573573
+ "Verify the <interactiveMode/> configuration in your settings.");
574574
return false;
575575
}
@@ -579,11 +579,12 @@ private boolean preCheck() throws MojoExecutionException {
579579
getLog().warn("");
580580
getLog().warn("All warranties with regard to this Maven goal are disclaimed!");
581581
getLog().warn("The changes will be done directly in the source code.");
582-
getLog().warn("The Maven Team strongly recommends the use of a SCM software BEFORE executing this goal.");
582+
getLog().warn(
583+
"The Maven Team strongly recommends committing the code to source code management BEFORE executing this goal.");
583584
getLog().warn("");
584585

585586
while (true) {
586-
getLog().info("Are you sure to proceed? [Y]es [N]o");
587+
getLog().info("Are you sure you want to proceed? [Y]es [N]o");
587588

588589
try {
589590
String userExpression = inputHandler.readLine();
@@ -592,7 +593,7 @@ private boolean preCheck() throws MojoExecutionException {
592593
break;
593594
}
594595
if (JavadocUtil.equalsIgnoreCase(userExpression, "N", "No")) {
595-
getLog().info("No changes in your sources occur.");
596+
getLog().info("OK, I will not change your source code.");
596597
return false;
597598
}
598599
} catch (IOException e) {
@@ -737,9 +738,9 @@ private void parseClirrTextOutputFile(File clirrTextOutputFile) throws IOExcepti
737738
}
738739
}
739740
if (clirrNewClasses.isEmpty() && clirrNewMethods.isEmpty()) {
740-
getLog().info("Clirr NOT found API differences.");
741+
getLog().info("Clirr did NOT find any API differences.");
741742
} else {
742-
getLog().info("Clirr found API differences, i.e. new classes/interfaces or methods.");
743+
getLog().info("Clirr found API differences; e.g. new classes, interfaces, or methods.");
743744
}
744745
}
745746

0 commit comments

Comments
 (0)