Respond to PR review comments

Author: Rachel Prince

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/BinaryType.java

@@ -16,6 +16,9 @@
 
 /** Enum of Android app binary types, used in {@link AppDistributionRelease} */
 public enum BinaryType {
+  /** Android App Bundle. */
   AAB,
+
+  /** Android Application Package. */
   APK
 }

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/FirebaseAppDistribution.java

@@ -140,15 +140,17 @@ public static FirebaseAppDistribution getInstance() {
   }
 
   /**
-   * Updates the app to the newest release, if one is available. Returns the release information or
-   * null if no update is found. Performs the following actions:
+   * Updates the app to the newest release, if one is available.
+   *
+   * <p>Returns the release information or null if no update is found. Performs the following
+   * actions:
    *
    * <ol>
-   *   <li>1. If tester is not signed in, presents the tester with a Google sign in UI
-   *   <li>2. Checks if a newer release is available. If so, presents the tester with a confirmation
+   *   <li>If tester is not signed in, presents the tester with a Google sign in UI.
+   *   <li>Checks if a newer release is available. If so, presents the tester with a confirmation
    *       dialog to begin the download.
-   *   <li>3. For APKs, downloads the binary and starts an installation intent.
-   *   <li>4. For AABs, directs the tester to the Play appto complete the download and installation.
+   *   <li>For APKs, downloads the binary and starts an installation intent. For AABs, directs the
+   *       tester to the Play app to complete the download and installation.
    * </ol>
    */
   @NonNull
@@ -256,7 +258,7 @@ public Task<Void> signInTester() {
 
   /**
    * Returns an {@link AppDistributionRelease} if an update is available for the current signed in
-   * tester. If no update is found, returns null.
+   * tester or {@code null} otherwise.
    */
   @NonNull
   public synchronized Task<AppDistributionRelease> checkForNewRelease() {
@@ -289,13 +291,13 @@ public synchronized Task<AppDistributionRelease> checkForNewRelease() {
   }
 
   /**
-   * 1:24 PM Updates app to the {@link AppDistributionRelease} returned by {@link
-   * #checkForNewRelease}. If the newest release is an APK, downloads the binary and starts an
-   * installation. If the newest release is an AAB, directs the tester to the Play app to complete
-   * the download and installation.
+   * Updates app to the {@link AppDistributionRelease} returned by {@link #checkForNewRelease}. If
+   * the newest release is an APK, downloads the binary and starts an installation. If the newest
+   * release is an AAB, directs the tester to the Play app to complete the download and
+   * installation.
    *
-   * <p>Cancels task with {@link FirebaseAppDistributionException.Status.UPDATE_NOT_AVAILABLE} if no
-   * new release is cached from {@link checkForNewRelease}.
+   * <p>Cancels task with {@link Status#UPDATE_NOT_AVAILABLE} if no new release is cached from
+   * {@link #checkForNewRelease}.
    */
   @NonNull
   public UpdateTask updateApp() {
@@ -337,7 +339,7 @@ private UpdateTask updateApp(boolean showDownloadInNotificationManager) {
     }
   }
 
-  /** Returns {@code true} if the App Distribution tester is signed in */
+  /** Returns {@code true} if the App Distribution tester is signed in. */
   public boolean isTesterSignedIn() {
     return this.signInStorage.getSignInStatus();
   }

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/FirebaseAppDistributionException.java

@@ -21,6 +21,7 @@
 
 /** The class for all Exceptions thrown by {@link FirebaseAppDistribution}. */
 public class FirebaseAppDistributionException extends FirebaseException {
+  /** Enum for potential error statuses that caused the {@link FirebaseAppDistributionException}. */
   public enum Status {
     /** Unknown error. */
     UNKNOWN,

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/UpdateProgress.java

@@ -28,20 +28,22 @@ static UpdateProgress.Builder builder() {
   }
 
   /**
-   * The number of bytes downloaded so far for an APK. Returns -1 if called when updating to an AAB
-   * or if no new release is available.
+   * The number of bytes downloaded so far for an APK.
+   *
+   * @returns -1 if called when updating to an AAB or if no new release is available.
    */
   @NonNull
   public abstract long getApkBytesDownloaded();
 
   /**
-   * The file size of the APK file to download in bytes. Returns -1 if called when updating to an
-   * AAB or if no new release is available.
+   * The file size of the APK file to download in bytes.
+   *
+   * @returns -1 if called when updating to an AAB or if no new release is available.
    */
   @NonNull
   public abstract long getApkFileTotalBytes();
 
-  /** Returns the current {@link UpdateStatus} of the update */
+  /** Returns the current {@link UpdateStatus} of the update. */
   @NonNull
   public abstract UpdateStatus getUpdateStatus();
 

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/UpdateStatus.java

@@ -28,19 +28,19 @@ public enum UpdateStatus {
   /** The new release failed to download. */
   DOWNLOAD_FAILED,
 
-  /** The new release installation was canceled */
+  /** The new release installation was canceled. */
   INSTALL_CANCELED,
 
-  /** The new release installation failed */
+  /** The new release installation failed. */
   INSTALL_FAILED,
 
-  /** The tester was redirected to Play to download an {@link BinaryType.AAB} file. */
+  /** The tester was redirected to Play to download an {@link BinaryType#AAB} file. */
   REDIRECTED_TO_PLAY,
 
   /** The tester is currently on the latest release they have access to for the current app. */
   NEW_RELEASE_NOT_AVAILABLE,
 
-  /** The call to {@link checkForNewRelease} failed. */
+  /** The call to {@link FirebaseAppDistribution#checkForNewRelease} failed. */
   NEW_RELEASE_CHECK_FAILED,
 
   /** The tester canceled the update. */

firebase-appdistribution/src/main/java/com/google/firebase/appdistribution/UpdateTask.java

@@ -20,8 +20,9 @@
 import java.util.concurrent.Executor;
 
 /**
- * Represents an asynchronous operation to update an app. This task also receives progress and other
- * state change notifications.
+ * Represents an asynchronous operation to update an app.
+ *
+ * <p>This task also receives progress and other state change notifications.
  */
 public abstract class UpdateTask extends Task<Void> {
   /**