Address rsimha feedback

ampproject · Sep 22, 2021 · 76b3762 · 76b3762
1 parent 402ae3d
commit 76b3762
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/build-system/tasks/helpers.js b/build-system/tasks/helpers.js
@@ -417,7 +417,7 @@ async function esbuildCompile(srcDir, srcFilename, destDir, options) {
     };
   }
   const {banner, footer} = splitWrapper();
-  const config = await getAmpConfigForFile(srcFilename, options);
+  const config = await getAmpConfigForFile(destFilename, options);
   const compiledFile = await getCompiledFile(srcFilename);
   banner.js = config + banner.js + compiledFile;
 

diff --git a/build-system/tasks/prepend-global/index.js b/build-system/tasks/prepend-global/index.js
@@ -198,7 +198,7 @@ async function getConfig(
  * @return {!Promise<string>}
  */
 async function getAmpConfigForFile(filename, options) {
-  const targets = MINIFIED_TARGETS.concat(UNMINIFIED_TARGETS);
+  const targets = options.minify ? MINIFIED_TARGETS : UNMINIFIED_TARGETS;
   const target = path.basename(filename, path.extname(filename));
   if (!!argv.noconfig || !targets.includes(target)) {
     return '';