Replace javadoc tags with Java Annotations (#165)

Co-authored-by: Marvin Froeder <[email protected]>

Author: velo

querydsl-maven-plugin/pom.xml

@@ -13,6 +13,10 @@
   <name>Querydsl - Maven plugin</name>
   <description>Querydsl Maven plugin</description>
 
+  <properties>
+    <maven-plugin.version>3.10.2</maven-plugin.version>
+  </properties>
+
   <dependencyManagement>
     <dependencies>
       <dependency>
@@ -34,20 +38,29 @@
   </dependencyManagement>
 
   <dependencies>
+    <dependency>
+      <groupId>org.apache.maven.plugin-tools</groupId>
+      <artifactId>maven-plugin-annotations</artifactId>
+      <version>${maven-plugin.version}</version>
+      <scope>provided</scope>
+    </dependency>
     <dependency>
       <groupId>org.apache.maven</groupId>
       <artifactId>maven-plugin-api</artifactId>
       <version>${mvn.version}</version>
+      <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>org.apache.maven</groupId>
       <artifactId>maven-model</artifactId>
       <version>${mvn.version}</version>
+      <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>org.apache.maven</groupId>
       <artifactId>maven-compat</artifactId>
       <version>${mvn.version}</version>
+      <scope>provided</scope>
     </dependency>
 
     <dependency>
@@ -96,37 +109,13 @@
   </dependencies>
 
   <build>
-    <pluginManagement>
-      <plugins>
-        <plugin>
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-plugin-plugin</artifactId>
-          <version>3.10.2</version>
-        </plugin>
-      </plugins>
-    </pluginManagement>
+    <plugins>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-plugin-plugin</artifactId>
+        <version>${maven-plugin.version}</version>
+      </plugin>
+    </plugins>
   </build>
 
-  <profiles>
-    <profile>
-      <!-- See http://stackoverflow.com/a/22296107/14731 -->
-      <id>doclint-java8-disable</id>
-      <activation>
-        <jdk>[1.8,)</jdk>
-      </activation>
-
-      <build>
-        <plugins>
-          <plugin>
-            <groupId>org.apache.maven.plugins</groupId>
-            <artifactId>maven-javadoc-plugin</artifactId>
-            <configuration>
-              <additionalparam>-Xdoclint:none</additionalparam>
-            </configuration>
-          </plugin>
-        </plugins>
-      </build>
-    </profile>
-  </profiles>
-
 </project>

querydsl-maven-plugin/src/main/java/com/querydsl/maven/AbstractExporterMojo.java

@@ -27,6 +27,8 @@
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.plugin.MojoFailureException;
+import org.apache.maven.plugins.annotations.Component;
+import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProject;
 import org.sonatype.plexus.build.incremental.BuildContext;
 
@@ -36,77 +38,43 @@
  */
 public abstract class AbstractExporterMojo extends AbstractMojo {
 
-  /**
-   * target folder for sources
-   *
-   * @parameter
-   * @required
-   */
+  /** maven project */
+  @Parameter(defaultValue = "${project}", required = true, readonly = true)
+  private MavenProject project;
+
+  /** target folder for sources */
+  @Parameter(required = true)
   private File targetFolder;
 
-  /**
-   * switch for scala source generation
-   *
-   * @parameter default-value=false
-   */
+  /** switch for scala source generation */
+  @Parameter(defaultValue = "false")
   private boolean scala;
 
-  /**
-   * packages to be exported
-   *
-   * @parameter
-   * @required
-   */
+  /** packages to be exported */
+  @Parameter(required = true)
   private String[] packages;
 
-  /**
-   * switch for inspecting fields
-   *
-   * @parameter default-value=true
-   */
+  /** switch for inspecting fields */
+  @Parameter(defaultValue = "true")
   private boolean handleFields = true;
 
-  /**
-   * switch for inspecting getters
-   *
-   * @parameter default-value=true
-   */
+  /** switch for inspecting getters */
+  @Parameter(defaultValue = "true")
   private boolean handleMethods = true;
 
-  /**
-   * switch for usage of field types instead of getter types
-   *
-   * @parameter default-value=false
-   */
+  /** switch for usage of field types instead of getter types */
+  @Parameter(defaultValue = "false")
   private boolean useFieldTypes = false;
 
-  /**
-   * maven project
-   *
-   * @parameter default-value="${project}"
-   * @readonly
-   */
-  private MavenProject project;
-
-  /**
-   * source file encoding
-   *
-   * @parameter
-   */
-  private String sourceEncoding;
+  /** source file encoding */
+  @Parameter private String sourceEncoding;
 
-  /**
-   * test classpath usage switch
-   *
-   * @parameter default-value=false
-   */
+  /** test classpath usage switch */
+  @Parameter(defaultValue = "false")
   private boolean testClasspath;
 
-  /**
-   * Whether to skip the exporting execution
-   *
-   * @parameter default-value=false property="maven.querydsl.skip"
-   */
+  /** Whether to skip the exporting execution */
+  @Parameter(defaultValue = "false", property = "maven.querydsl.skip")
   private boolean skip;
 
   /**
@@ -116,17 +84,11 @@ public abstract class AbstractExporterMojo extends AbstractMojo {
    * <em>See also</em> <a
    * href="https://docs.oracle.com/javase/specs/jls/se8/html/jls-9.html#jls-9.7.3">Single-Element
    * Annotation</a>
-   *
-   * @parameter
    */
-  private String generatedAnnotationClass;
+  @Parameter private String generatedAnnotationClass;
 
-  /**
-   * build context
-   *
-   * @component
-   */
-  private BuildContext buildContext;
+  /** build context */
+  @Component private BuildContext buildContext;
 
   @SuppressWarnings("unchecked")
   @Override