Avoid using Promise.all to iterate through paths

Originally, this was added to ensure maximum parallelism and
speed. But we are seeing out of memory errors on larger
repositories.

This changes avoids using Promise.all and instead uses a for loop.

src/directory.ts

@@ -166,15 +166,17 @@ export class LocDir {
       };
     } = {};
 
-    await Promise.all(paths.map(async (pathItem) => {
+    // We _could_ use Promise.all to count the files in parallel, but that
+    // would lead to out of memory errors when there are many files.
+    for (const pathItem of paths) {
       const fullPath = slash(path.join(this.cwd, pathItem));
       if (
         !pathItem ||
         this.ignoreLanguage(pathItem) ||
         !(await fs.pathExists(fullPath)) ||
         (await fs.stat(fullPath)).isDirectory()
       ) {
-        return;
+        continue;
       }
       const file = new LocFile(fullPath);
       const fileLineInfo = await file.getFileInfo();
@@ -192,7 +194,7 @@ export class LocDir {
         [fileLineInfo.languages]: language,
       };
       files.push(fullPath);
-    }));
+    }
 
     return {
       files,