desruisseaux commented on code in PR #1017:
URL:
https://github.com/apache/maven-compiler-plugin/pull/1017#discussion_r2729466280
##########
src/main/java/org/apache/maven/plugin/compiler/AbstractCompilerMojo.java:
##########
@@ -83,39 +83,44 @@
import static org.apache.maven.plugin.compiler.SourceDirectory.MODULE_INFO;
/**
- * Base class of Mojos compiling Java source code.
- * This plugin uses the {@link JavaCompiler} interface from JDK 6+.
- * Each instance shall be used only once, then discarded.
+ * Base class of Mojos compiling Java source code. This plugin uses the
Review Comment:
I would add that the previous formatting was intentional. We try to change
lines after a natural separator such as a dot, or a comma if the line is too
long. This is because in commits, changes are shown on a line-by-line basis.
When sentences are on their own line, then if (for example) the sentence "Base
class of Mojos…" is changed, it is easier to see that the "This plugin uses…"
sentence is not modified.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
