desruisseaux commented on code in PR #481:
URL: https://github.com/apache/maven-jar-plugin/pull/481#discussion_r2425499707
##########
src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java:
##########
@@ -158,76 +145,79 @@ public abstract class AbstractJarMojo implements
org.apache.maven.api.plugin.Moj
private String outputTimestamp;
/**
+ * Whether to detect multi-release JAR files.
* If the JAR contains the {@code META-INF/versions} directory it will be
detected as a multi-release JAR file
- * ("MRJAR"), adding the {@code Multi-Release: true} attribute to the main
section of the JAR MANIFEST.MF.
+ * ("MRJAR"), adding the {@code Multi-Release: true} attribute to the main
section of the JAR {@code MANIFEST.MF}.
*
* @since 3.4.0
*/
@Parameter(property = "maven.jar.detectMultiReleaseJar", defaultValue =
"true")
private boolean detectMultiReleaseJar;
/**
- * The mojo logger
+ * The <abbr>MOJO</abbr> logger.
*/
@Inject
private Log log;
/**
- * Return the specific output directory to serve as the root for the
archive.
- * @return get classes directory.
+ * Creates a new <abbr>MOJO</abbr>.
+ */
+ protected AbstractJarMojo() {}
+
+ /**
+ * {@return the specific output directory to serve as the root for the
archive}.
*/
protected abstract Path getClassesDirectory();
/**
- * Return the {@link #project MavenProject}
+ * Return the {@linkplain #project Maven project}.
*
- * @return the MavenProject.
+ * @return the Maven project
*/
protected final Project getProject() {
return project;
}
+ /**
+ * {@return the <abbr>MOJO</abbr> logger}.
Review Comment:
Just tested and I confirm. With an explicit dot after `}`, the `javadoc`
tools generates a sentence ending with 2 dots. I will need to pass over the
other plugins since I have used the `{@return …}.` form in many places.
--
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]
