remove esbuild config (redwoodjs#3541)

Co-authored-by: David Price <thedavid@thedavidprice.com>
YannBirba · Oct 16, 2021 · d788d79 · d788d79
1 parent 0446a59
commit d788d79
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 31 deletions.
diff --git a/packages/codemods/src/lib/getRWPaths.ts b/packages/codemods/src/lib/getRWPaths.ts
@@ -47,9 +47,6 @@ interface Config {
     stories: boolean
     nestScaffoldByModel: boolean
   }
-  experimental: {
-    esbuild: boolean
-  }
 }
 
 // Note that web's includeEnvironmentVariables is handled in `webpack.common.js`
@@ -82,9 +79,6 @@ const DEFAULT_CONFIG: Config = {
     stories: true,
     nestScaffoldByModel: true,
   },
-  experimental: {
-    esbuild: false,
-  },
 }
 
 /**

diff --git a/packages/core/config/webpack.common.js b/packages/core/config/webpack.common.js
@@ -169,8 +169,6 @@ module.exports = (webpackEnv) => {
   const shouldIncludeFastRefresh =
     redwoodConfig.web.experimentalFastRefresh && !isEnvProduction
 
-  const shouldUseEsbuild = process.env.ESBUILD === '1'
-
   return {
     mode: isEnvProduction ? 'production' : 'development',
     devtool: isEnvProduction ? 'source-map' : 'cheap-module-source-map',
@@ -274,12 +272,6 @@ module.exports = (webpackEnv) => {
                     ].filter(Boolean),
                   },
                 },
-                shouldUseEsbuild && {
-                  loader: 'esbuild-loader',
-                  options: {
-                    loader: 'jsx',
-                  },
-                },
               ].filter(Boolean),
             },
             // (2)
@@ -297,12 +289,6 @@ module.exports = (webpackEnv) => {
                     ].filter(Boolean),
                   },
                 },
-                shouldUseEsbuild && {
-                  loader: 'esbuild-loader',
-                  options: {
-                    loader: 'tsx',
-                  },
-                },
               ].filter(Boolean),
             },
             // .module.css (3), .css (4), .module.scss (5), .scss (6)

diff --git a/packages/internal/src/__tests__/config.test.ts b/packages/internal/src/__tests__/config.test.ts
@@ -20,9 +20,6 @@ describe('getConfig', () => {
         "browser": Object {
           "open": false,
         },
-        "experimental": Object {
-          "esbuild": false,
-        },
         "generate": Object {
           "nestScaffoldByModel": true,
           "stories": true,

diff --git a/packages/internal/src/config.ts b/packages/internal/src/config.ts
@@ -47,9 +47,6 @@ export interface Config {
     stories: boolean
     nestScaffoldByModel: boolean
   }
-  experimental: {
-    esbuild: boolean
-  }
 }
 
 // Note that web's includeEnvironmentVariables is handled in `webpack.common.js`
@@ -82,9 +79,6 @@ const DEFAULT_CONFIG: Config = {
     stories: true,
     nestScaffoldByModel: true,
   },
-  experimental: {
-    esbuild: false,
-  },
 }
 
 /**

diff --git a/tasks/e2e/cypress/integration/01-tutorial/codemods/Step0_1_RedwoodToml.js b/tasks/e2e/cypress/integration/01-tutorial/codemods/Step0_1_RedwoodToml.js
@@ -13,6 +13,4 @@ export default `
   port = 8911
 [browser]
   open = true
-[experimental]
-  esbuild = false
 `