memoize in place

Author: thymikee

packages/platform-ios/src/config/findProject.js

@@ -10,7 +10,6 @@
 
 import glob from 'glob';
 import path from 'path';
-import {memoize} from 'lodash';
 
 /**
  * Glob pattern to look for xcodeproj
@@ -38,10 +37,9 @@ const GLOB_EXCLUDE_PATTERN = ['**/@(Pods|node_modules)/**'];
  *
  * Returns first match if files are found or null
  *
- * Note: `./ios/*.xcodeproj` are returned regardless of the name.
- * Note2: Globbing is expensive and often repeated on the same folder, hence memoizing
+ * Note: `./ios/*.xcodeproj` are returned regardless of the name
  */
-export default memoize(function findProject(folder: string): string | null {
+export default function findProject(folder: string): string | null {
   const projects = glob
     .sync(GLOB_PATTERN, {
       cwd: folder,
@@ -58,4 +56,4 @@ export default memoize(function findProject(folder: string): string | null {
   }
 
   return projects[0];
-});
+}

packages/platform-ios/src/config/index.js

@@ -9,11 +9,14 @@
  */
 
 import path from 'path';
+import {memoize} from 'lodash';
 import findProject from './findProject';
 import findPodfilePath from './findPodfilePath';
 import findPodspec from './findPodspec';
 import type {UserConfigT} from 'types';
 
+const memoizedFindProject = memoize(findProject);
+
 /**
  * For libraries specified without an extension, add '.tbd' for those that
  * start with 'lib' and '.framework' to the rest.
@@ -37,7 +40,7 @@ export function projectConfig(
   if (!userConfig) {
     return;
   }
-  const project = userConfig.project || findProject(folder);
+  const project = userConfig.project || memoizedFindProject(folder);
 
   /**
    * No iOS config found here