ignore build directory in default build (#874)

FredKSchott · Aug 22, 2020 · ee05c28 · ee05c28 · vercel · Aug 22, 2020
1 parent 5b0fe76
commit ee05c28
Show file tree

Hide file tree

Showing 8 changed files with 41 additions and 2 deletions.
diff --git a/packages/snowpack/src/config.ts b/packages/snowpack/src/config.ts
@@ -546,7 +546,7 @@ function normalizeConfig(config: SnowpackConfig): SnowpackConfig {
   config.knownEntrypoints = (config as any).install || [];
   config.installOptions.dest = path.resolve(cwd, config.installOptions.dest);
   config.devOptions.out = path.resolve(cwd, config.devOptions.out);
-  config.exclude = Array.from(new Set([...ALWAYS_EXCLUDE, ...config.exclude]));
+  config.exclude = Array.from(new Set([...ALWAYS_EXCLUDE, `${config.devOptions.out}/**/*`, ...config.exclude]));
 
   if (!config.proxy) {
     config.proxy = {} as any;

diff --git a/test/build/__snapshots__/build.test.js.snap b/test/build/__snapshots__/build.test.js.snap
diff --git a/test/build/build.test.js b/test/build/build.test.js
@@ -26,7 +26,7 @@ describe('snowpack build', () => {
 
       // build test
       execa.sync('yarn', ['testbuild'], {cwd});
-      const actual = path.join(cwd, 'build');
+      const actual = testName === 'config-out' ? path.join(cwd, 'TEST_BUILD_OUT') : path.join(cwd, 'build');
 
       // Test That all files match
       const allFiles = glob.sync(`**/*`, {

diff --git a/test/build/config-out/TEST_BUILD_OUT/__snowpack__/env.js b/test/build/config-out/TEST_BUILD_OUT/__snowpack__/env.js
@@ -0,0 +1 @@
+export default {"MODE":"production","NODE_ENV":"production"};
diff --git a/test/build/config-out/TEST_BUILD_OUT/package.json b/test/build/config-out/TEST_BUILD_OUT/package.json
@@ -0,0 +1,18 @@
+{
+  "private": true,
+  "version": "1.0.1",
+  "name": "@snowpack/test-config-out",
+  "description": "Test for devOptions.out",
+  "scripts": {
+    "testbuild": "snowpack build"
+  },
+  "snowpack": {
+    "devOptions": {
+      "out": "TEST_BUILD_OUT"
+    }
+  },
+  "devDependencies": {
+    "cross-env": "^7.0.2",
+    "snowpack": "^2.7.0"
+  }
+}
diff --git a/test/build/config-out/TEST_BUILD_OUT/src/index.js b/test/build/config-out/TEST_BUILD_OUT/src/index.js
@@ -0,0 +1 @@
+console.log("fooey");
diff --git a/test/build/config-out/package.json b/test/build/config-out/package.json
@@ -0,0 +1,18 @@
+{
+  "private": true,
+  "version": "1.0.1",
+  "name": "@snowpack/test-config-out",
+  "description": "Test for devOptions.out",
+  "scripts": {
+    "testbuild": "snowpack build"
+  },
+  "snowpack": {
+    "devOptions": {
+      "out": "TEST_BUILD_OUT"
+    }
+  },
+  "devDependencies": {
+    "cross-env": "^7.0.2",
+    "snowpack": "^2.7.0"
+  }
+}
diff --git a/test/build/config-out/src/index.js b/test/build/config-out/src/index.js
@@ -0,0 +1 @@
+console.log('fooey');