throw an error if anyone sets skipBabel

packages/compat/src/options.ts

@@ -141,6 +141,12 @@ export function optionsWithDefaults(options?: Options): CompatOptionsType {
     );
   }
 
+  if ((options as any)?.skipBabel !== undefined) {
+    throw new Error(
+      `You have set 'skipBabel' on your Embroider options. This setting has been removed and you can now configure your babel ignores directly in the babel config in your repo https://babeljs.io/docs/options#ignore`
+    );
+  }
+
   return Object.assign({}, defaults, options);
 }
 

tests/addon-template/ember-cli-build.js

@@ -15,11 +15,5 @@ module.exports = function (defaults) {
   */
 
   const { maybeEmbroider } = require('@embroider/test-setup');
-  return maybeEmbroider(app, {
-    skipBabel: [
-      {
-        package: 'qunit',
-      },
-    ],
-  });
+  return maybeEmbroider(app);
 };

tests/fixtures/blacklisted-addon-build-options/ember-cli-build.js

@@ -10,12 +10,5 @@ module.exports = function (defaults) {
     },
   });
 
-  return maybeEmbroider(app, {
-
-    skipBabel: [
-      {
-        package: 'qunit',
-      },
-    ],
-  });
+  return maybeEmbroider(app);
 };

tests/fixtures/macro-sample-addon/ember-cli-build.js

@@ -18,10 +18,5 @@ module.exports = function(defaults) {
   return maybeEmbroider(app, {
     useAddonAppBoot: false,
     useAddonConfigModule: false,
-    skipBabel: [
-      {
-        package: 'qunit',
-      },
-    ],
   });
 };

tests/scenarios/compat-addon-classic-features-test.ts

@@ -52,11 +52,6 @@ appScenarios
 
           return maybeEmbroider(app, {
             availableContentForTypes: ['custom'],
-            skipBabel: [
-              {
-                package: 'qunit',
-              },
-            ],
           });
         };
       `,

tests/scenarios/compat-exclude-dot-files-test.ts

@@ -23,11 +23,6 @@ appScenarios
 
         return maybeEmbroider(app, {
           staticAddonTrees: false,
-          skipBabel: [
-            {
-              package: 'qunit',
-            },
-          ],
         });
       };
       `,

tests/scenarios/router-test.ts

@@ -36,11 +36,6 @@ function setupScenario(project: Project) {
             staticAddonTrees: true,
             staticInvokables: true,
             splitAtRoutes: ['split-me'],
-            skipBabel: [
-              {
-                package: 'qunit',
-              },
-            ],
           });
         };
       `,

tests/scenarios/stage1-test.ts

@@ -30,11 +30,6 @@ appScenarios
             staticComponents: false,
             staticHelpers: false,
             staticModifiers: false,
-            skipBabel: [
-              {
-                package: 'qunit',
-              },
-            ],
           });
         };
       `,

tests/scenarios/static-app-test.ts

@@ -390,7 +390,6 @@ wideAppScenarios
                 },
               },
             ],
-            skipBabel: [{ package: 'qunit' }, { package: 'macro-decorators' }],
           });
         };
 

tests/ts-app-template-classic/ember-cli-build.js

@@ -8,11 +8,5 @@ module.exports = function (defaults) {
     'ember-cli-babel': { enableTypeScriptTransform: true },
   });
 
-  return maybeEmbroider(app, {
-    skipBabel: [
-      {
-        package: 'qunit',
-      },
-    ],
-  });
+  return maybeEmbroider(app);
 };

tests/ts-app-template/ember-cli-build.js

@@ -8,11 +8,5 @@ module.exports = function (defaults) {
     'ember-cli-babel': { enableTypeScriptTransform: true },
   });
 
-  return maybeEmbroider(app, {
-    skipBabel: [
-      {
-        package: 'qunit',
-      },
-    ],
-  });
+  return maybeEmbroider(app);
 };