disable default features for turbopack-ecmascript-plugins (vercel/tur…

…borepo#5363) ### Description it breaks no-plugin in next.js
vercel · Jun 22, 2023 · c06f012 · c06f012
1 parent 024f706
commit c06f012
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/crates/turbopack-binding/Cargo.toml b/crates/turbopack-binding/Cargo.toml
@@ -209,7 +209,7 @@ turbopack-css = { optional = true, workspace = true }
 turbopack-dev = { optional = true, workspace = true }
 turbopack-dev-server = { optional = true, workspace = true }
 turbopack-ecmascript = { optional = true, workspace = true }
-turbopack-ecmascript-plugins = { optional = true, workspace = true }
+turbopack-ecmascript-plugins = { optional = true, workspace = true, default-features = false }
 turbopack-ecmascript-runtime = { optional = true, workspace = true }
 turbopack-env = { optional = true, workspace = true }
 turbopack-image = { optional = true, workspace = true }

diff --git a/crates/turbopack-ecmascript-plugins/src/transform/swc_ecma_transform_plugins.rs b/crates/turbopack-ecmascript-plugins/src/transform/swc_ecma_transform_plugins.rs
@@ -45,6 +45,8 @@ impl SwcPluginModule {
 
         #[cfg(not(feature = "swc_ecma_transform_plugin"))]
         {
+            let _ = plugin_name;
+            let _ = plugin_bytes;
             Self(())
         }
     }
@@ -105,6 +107,7 @@ impl SwcEcmaTransformPluginsTransformer {
     // [TODO] Due to WEB-1102 putting this module itself behind compile time feature
     // doesn't work. Instead allow to instantiate dummy instance.
     #[cfg(not(feature = "swc_ecma_transform_plugin"))]
+    #[allow(clippy::new_without_default)]
     pub fn new() -> Self {
         Self {}
     }