refactor: Remove unnecessary sass-loader compat

preactjs · Dec 14, 2022 · f291e6c · f291e6c
1 parent ef58367
commit f291e6c
Showing 1 changed file with 6 additions and 14 deletions.
diff --git a/packages/cli/src/lib/webpack/webpack-base-config.js b/packages/cli/src/lib/webpack/webpack-base-config.js
@@ -45,19 +45,6 @@ function resolveTsconfig(cwd, isProd) {
 	}
 }
 
-function getSassConfiguration(...includePaths) {
-	const config = {
-		sourceMap: true,
-		sassOptions: {
-			includePaths,
-		},
-	};
-
-	Object.defineProperty(config, 'includePaths', { value: includePaths });
-
-	return config;
-}
-
 /**
  * @param {import('../../../types').Env} env
  * @returns {import('webpack').Configuration}
@@ -185,7 +172,12 @@ module.exports = function createBaseConfig(config, env) {
 							options: {
 								cwd,
 								loader: tryResolveOptionalLoader('sass-loader'),
-								options: getSassConfiguration(...nodeModules),
+								options: {
+									sourceMap: true,
+									sassOptions: {
+										includePaths: [...nodeModules],
+									},
+								},
 							},
 						},
 					],