Update comments after review

Author: kjelko

packages/remote-config/src/api.ts

@@ -60,7 +60,7 @@ export function getRemoteConfig(
 
   if (options.initialFetchResponse) {
     // We use these initial writes as the initialization promise since they will hydrate the same
-    // fields that storageCache.loadFromStorage would set.
+    // fields that `storageCache.loadFromStorage` would set.
     rc._initializePromise = Promise.all([
       rc._storage.setLastSuccessfulFetchResponse(options.initialFetchResponse),
       rc._storage.setActiveConfigEtag(options.initialFetchResponse?.eTag || ''),
@@ -70,7 +70,7 @@ export function getRemoteConfig(
         options.initialFetchResponse?.config || {}
       )
     ]).then();
-    // The storageCache methods above set their in-memory fields sycnhronously, so it's
+    // The `storageCache` methods above set their in-memory fields synchronously, so it's
     // safe to declare our initialization complete at this point.
     rc._isInitializationComplete = true;
   }
@@ -254,7 +254,7 @@ export function getValue(remoteConfig: RemoteConfig, key: string): Value {
   if (!rc._isInitializationComplete) {
     rc._logger.debug(
       `A value was requested for key "${key}" before SDK initialization completed.` +
-        ' Await on ensureInitialized if the intent was to get a previously activated value.'
+      ' Await on ensureInitialized if the intent was to get a previously activated value.'
     );
   }
   const activeConfig = rc._storageCache.getActiveConfig();
@@ -265,7 +265,7 @@ export function getValue(remoteConfig: RemoteConfig, key: string): Value {
   }
   rc._logger.debug(
     `Returning static value for key "${key}".` +
-      ' Define a default or remote value if this is unintentional.'
+    ' Define a default or remote value if this is unintentional.'
   );
   return new ValueImpl('static');
 }

packages/remote-config/src/public_types.ts

@@ -60,15 +60,15 @@ export interface FirebaseRemoteConfigObject {
 /**
  * Defines a successful response (200 or 304).
  *
- * <p>Modeled after the native {@link Response} interface, but simplified for Remote Config's
+ * <p>Modeled after the native `Response` interface, but simplified for Remote Config's
  * use case.
  */
 export interface FetchResponse {
   /**
    * The HTTP status, which is useful for differentiating success responses with data from
    * those without.
    *
-   * <p>{@link RemoteConfigClient} is modeled after the native {@link GlobalFetch} interface, so
+   * <p>The Remote Config client is modeled after the native `Fetch` interface, so
    * HTTP status is first-class.
    *
    * <p>Disambiguation: the fetch response returns a legacy "state" value that is redundant with the