Use for of instead for

Author: Carlos Lancha

src/config.ts

@@ -32,15 +32,13 @@ export function validateConfig(config: Config): Config {
     throw new Error('callToImport is not a valid config array.');
   }
 
-  for (let i=0; i < config.callToImport.length; i++) {
-    let callToImportItem = config.callToImport[i];
-
-    if (!isRegex(callToImportItem.regex)) {
-      throw new Error(`callToImport.regex "${callToImportItem.regex}" is not a valid RegExp.`);
+  for (const item of config.callToImport) {
+    if (!isRegex(item.regex)) {
+      throw new Error(`callToImport.regex "${item.regex}" is not a valid RegExp.`);
     }
 
-    if (!isRegex(callToImportItem.replace)) {
-      throw new Error(`callToImport.replace "${callToImportItem.replace}" is not a valid replace string.`);
+    if (!isRegex(item.replace)) {
+      throw new Error(`callToImport.replace "${item.replace}" is not a valid replace string.`);
     }
   }
 

src/validate-call-imports.ts

@@ -40,8 +40,8 @@ export default function valdiateCallImports(soyContext: SoyContext, jsContext: J
 
     const missingImports = getExternalSoyCalls(soyContext)
     .filter(name => {
-      for (let i=0; i < config.callToImport.length; i++) {
-        let transformedName = transform(name, config.callToImport[i].regex, config.callToImport[i].replace);
+      for (const item of config.callToImport) {
+        let transformedName = transform(name, item.regex, item.replace);
 
         if (importNames.find(importName => importName.includes(transformedName))) {
           return false;