cosmetic: eliminate code warnings

Author: stefanseifert

tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/AbstractContentPackageMojo.java

@@ -44,9 +44,8 @@ abstract class AbstractContentPackageMojo extends AbstractMojo {
   private static final String CONSOLE_URL = "/system/console";
 
   /**
-   * <p>
    * The URL of the HTTP service API of the CRX package manager.
-   * </p>
+   *
    * <p>
    * See <a href=
    * "http://dev.day.com/docs/en/crx/current/how_to/package_manager.html#Managing%20Packages%20on%20the%20Command%20Line"
@@ -116,17 +115,18 @@ abstract class AbstractContentPackageMojo extends AbstractMojo {
   private int retryDelay;
 
   /**
-   * <p>
    * Bundle status JSON URL. If an URL is configured the activation status of all bundles in the system is checked
    * before it is tried to upload and install a new package and after each upload.
-   * </p>
+   *
    * <p>
    * If not all bundles are activated the upload is delayed up to {@link #bundleStatusWaitLimit} seconds,
    * every 5 seconds the activation status is checked anew.
    * </p>
+   *
    * <p>
    * Expected is an URL like: http://localhost:4502/system/console/bundles/.json
    * </p>
+   *
    * <p>
    * If the URL is not set it is derived from serviceURL. If set to "-" the status check is disabled.
    * </p>
@@ -142,17 +142,18 @@ abstract class AbstractContentPackageMojo extends AbstractMojo {
   private int bundleStatusWaitLimit;
 
   /**
-   * <p>
    * Package Manager install status JSON URL. If an URL is configured the installation status of packages and
    * embedded packages is checked before it is tried to upload and install a new package and after each upload.
-   * </p>
+   *
    * <p>
    * If not all packages are installed the upload is delayed up to {@link #packageManagerInstallStatusWaitLimit}
    * seconds, every 5 seconds the installation status is checked anew.
    * </p>
+   *
    * <p>
    * Expected is an URL like: http://localhost:4502/crx/packmgr/installstatus.jsp
    * </p>
+   *
    * <p>
    * If the URL is not set it is derived from serviceURL. If set to "-" the status check is disabled.
    * </p>

tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/CloudManagerAllPackageMojo.java

@@ -44,11 +44,13 @@
 /**
  * Builds an "all" content package dedicated for deployment via Adobe Cloud Manager
  * for the given environment and node(s).
+ *
  * <p>
  * By default, it builds one "all" package per environment and node without adding any Cloud Manager
  * environment-specific run mode suffixes to the folders. By defining a parameter <code>cloudManager.target</code>
  * (contains a list of string values) in the CONGA environment it is possible:
  * </p>
+ *
  * <ul>
  * <li>If it contains <code>none</code> no "all" package is build.</li>
  * <li>If set to one or multiple environment names (normally dev/stage/prod) one "all" package for each of
@@ -85,6 +87,7 @@ public final class CloudManagerAllPackageMojo extends AbstractCloudManagerMojo {
 
   /**
    * Automatically generate dependencies between content packages based on file order in CONGA configuration.
+   *
    * <p>
    * Possible options see
    * <a href="apidocs/io/wcm/devops/conga/plugins/aem/maven/AutoDependenciesMode.html">AutoDependenciesMode</a>.
@@ -95,6 +98,7 @@ public final class CloudManagerAllPackageMojo extends AbstractCloudManagerMojo {
 
   /**
    * How to optimize author/publish run modes in resulting "all" package.
+   *
    * <p>
    * Possible options see
    * <a href="apidocs/io/wcm/devops/conga/plugins/aem/maven/RunModeOptimization.html">RunModeOptimization</a>.
@@ -105,6 +109,7 @@ public final class CloudManagerAllPackageMojo extends AbstractCloudManagerMojo {
 
   /**
    * How to validate package types to be included in "all" package.
+   *
    * <p>
    * Possible options see
    * <a href="apidocs/io/wcm/devops/conga/plugins/aem/maven/PackageTypeValidation.html">PackageTypeValidation</a>.
@@ -115,6 +120,7 @@ public final class CloudManagerAllPackageMojo extends AbstractCloudManagerMojo {
 
   /**
    * How to handle versions of packages and sub-packages inside "all" package.
+   *
    * <p>
    * Possible options see
    * <a href="apidocs/io/wcm/devops/conga/plugins/aem/maven/PackageVersionMode.html">PackageVersionMode</a>.

tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/PackageVersionMode.java

@@ -33,9 +33,11 @@ public enum PackageVersionMode {
    * Suffix the version number of all packages with a release version with the version of the POM the Mojo runs in.
    * Within the version suffix, dots are replaced with underlines to avoid convision with the main version number.
    * This is useful when deploying to AMS with Cloud Manager.
+   *
    * <p>
    * Example:
    * </p>
+   *
    * <ul>
    * <li>Original package version: 2.5.0</li>
    * <li>POM version: 2022.1103.152749.0000000571</li>

tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/AllPackageBuilder.java

@@ -80,9 +80,11 @@
 
 /**
  * Builds "all" package based on given set of content packages.
+ *
  * <p>
  * General concept:
  * </p>
+ *
  * <ul>
  * <li>Iterates through all content packages that are generated or collected by CONGA and contained in the
  * model.json</li>