fix: remove extra promise allocation

sidwebworks · Jul 19, 2022 · a6eb172 · a6eb172
1 parent c687b65
commit a6eb172
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -25,10 +25,10 @@ function createValidator(workbook: WorkBook, opts?: ValidatorOptions) {
     header: options.header,
   })
 
-  const parse = async (row: any, schema: ZodSchema) => {
+  const parse = (row: any, schema: ZodSchema) => {
     const data = toObject(row, header)
     try {
-      await schema.parseAsync(data)
+      schema.parse(data)
       options.onValid && options.onValid(data)
       return { issues: [], isValid: true, data }
     } catch (error) {
@@ -40,10 +40,8 @@ function createValidator(workbook: WorkBook, opts?: ValidatorOptions) {
     }
   }
 
-  const validate = async (schema: ZodSchema): Promise<Result> => {
-    const promises = rows.map((row) => parse(row, schema))
-
-    const result = await Promise.all(promises)
+  const validate = (schema: ZodSchema): Result => {
+    const result = rows.map((row) => parse(row, schema))
 
     return {
       valid: result.filter((r) => r.isValid),