Remove unnecessary comment

elastic · Jul 29, 2024 · 03c9209 · 03c9209
1 parent 5080816
commit 03c9209
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/plugins/discover/public/embeddable/actions/view_saved_search_action.ts b/src/plugins/discover/public/embeddable/actions/view_saved_search_action.ts
@@ -50,7 +50,7 @@ export class ViewSavedSearchAction implements Action<EmbeddableApiContext> {
     const hasDiscoverPermissions =
       (capabilities.discover.show as boolean) || (capabilities.discover.save as boolean);
 
-    if (!hasDiscoverPermissions) return false;
+    if (!hasDiscoverPermissions) return false; // early return to delay async import until absolutely necessary
     const { compatibilityCheck } = await import('./view_saved_search_compatibility_check');
     return compatibilityCheck(embeddable);
   }

diff --git a/src/plugins/discover/public/embeddable/actions/view_saved_search_compatibility_check.ts b/src/plugins/discover/public/embeddable/actions/view_saved_search_compatibility_check.ts
@@ -22,7 +22,6 @@ import { apiPublishesSavedSearch, PublishesSavedSearch } from '../types';
 
 type ViewSavedSearchActionApi = CanAccessViewMode & HasType & PublishesSavedSearch;
 
-/** Async type guards aren't supported, so need to make this an async getter instead */
 export const compatibilityCheck = (
   api: EmbeddableApiContext['embeddable']
 ): api is ViewSavedSearchActionApi => {