Simplify invalidateOnFileChange() in Transformers (#9379)

Co-authored-by: Niklas Mischkulnig <4586894+mischnic@users.noreply.github.com>
parcel-bundler · Nov 13, 2023 · c88116a · c88116a
1 parent c784885
commit c88116a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 10 deletions.
diff --git a/packages/transformers/posthtml/src/PostHTMLTransformer.js b/packages/transformers/posthtml/src/PostHTMLTransformer.js
@@ -105,14 +105,11 @@ export default (new Transformer({
     });
 
     if (res.messages) {
-      await Promise.all(
-        res.messages.map(({type, file: filePath}) => {
-          if (type === 'dependency') {
-            return asset.invalidateOnFileChange(filePath);
-          }
-          return Promise.resolve();
-        }),
-      );
+      for (let {type, file: filePath} of res.messages) {
+        if (type === 'dependency') {
+          asset.invalidateOnFileChange(filePath);
+        }
+      }
     }
 
     asset.setAST({

diff --git a/packages/transformers/pug/src/PugTransformer.js b/packages/transformers/pug/src/PugTransformer.js
@@ -31,7 +31,7 @@ export default (new Transformer({
     });
 
     for (let filePath of render.dependencies) {
-      await asset.invalidateOnFileChange(filePath);
+      asset.invalidateOnFileChange(filePath);
     }
 
     asset.type = 'html';

diff --git a/packages/transformers/stylus/src/StylusTransformer.js b/packages/transformers/stylus/src/StylusTransformer.js
@@ -198,7 +198,7 @@ async function getDependencies(
       // Recursively process resolved files as well to get nested deps
       for (let resolved of found) {
         if (!seen.has(resolved)) {
-          await asset.invalidateOnFileChange(resolved);
+          asset.invalidateOnFileChange(resolved);
 
           let code = await asset.fs.readFile(resolved, 'utf8');
           for (let [path, resolvedPath] of await getDependencies(