refactor: 💡 remove more deprecated things

Author: kaisermann

src/autoProcess.ts

@@ -7,39 +7,16 @@ import {
   Processed,
   TransformerArgs,
   TransformerOptions,
+  Transformers,
 } from './types';
 import { hasPostCssInstalled } from './modules/hasPostcssInstalled';
 import { concat } from './modules/concat';
 import { parseFile } from './modules/parseFile';
 import { addLanguageAlias } from './modules/language';
 import { throwError } from './modules/errors';
 
-interface Transformers {
-  typescript?: TransformerOptions<Options.Typescript>;
-  scss?: TransformerOptions<Options.Sass>;
-  sass?: TransformerOptions<Options.Sass>;
-  less?: TransformerOptions<Options.Less>;
-  stylus?: TransformerOptions<Options.Stylus>;
-  postcss?: TransformerOptions<Options.Postcss>;
-  coffeescript?: TransformerOptions<Options.Coffeescript>;
-  pug?: TransformerOptions<Options.Pug>;
-  globalStyle?: Options.GlobalStyle;
-  replace?: Options.Replace;
-  [languageName: string]: TransformerOptions;
-}
-
-type AutoPreprocessOptions = {
-  /** @deprecated for svelte v3 use instead a array of processors */
-  onBefore?: ({
-    content,
-    filename,
-  }: {
-    content: string;
-    filename: string;
-  }) => Promise<string> | string;
+type AutoPreprocessOptions = Transformers & {
   markupTagName?: string;
-  /** @deprecated add transformer config directly to svelte-preprocess options object */
-  transformers?: Transformers;
   aliases?: Array<[string, string]>;
   preserve?: string[];
   typescript?: TransformerOptions<Options.Typescript>;

src/types/index.ts

@@ -38,3 +38,16 @@ export type TransformerOptions<T = any> =
   | boolean
   | Record<keyof T | string, T[keyof T] | any>
   | Transformer<T>;
+
+export interface Transformers {
+  typescript?: TransformerOptions<Options.Typescript>;
+  scss?: TransformerOptions<Options.Sass>;
+  sass?: TransformerOptions<Options.Sass>;
+  less?: TransformerOptions<Options.Less>;
+  stylus?: TransformerOptions<Options.Stylus>;
+  postcss?: TransformerOptions<Options.Postcss>;
+  coffeescript?: TransformerOptions<Options.Coffeescript>;
+  pug?: TransformerOptions<Options.Pug>;
+  globalStyle?: Options.GlobalStyle;
+  replace?: Options.Replace;
+}

test/autoProcess/markup.test.ts

@@ -45,10 +45,10 @@ MARKUP_LANGS.forEach(([lang, ext]) => {
       `template.${ext}`,
     )}</template>`;
 
-    it(`should throw parsing ${lang} when { ${lang}: false }`, async () => {
+    it(`should NOT throw parsing ${lang} when { ${lang}: false }`, async () => {
       const opts = getAutoPreprocess({ pug: false });
 
-      expect(await doesCompileThrow(template, opts)).toBe(true);
+      expect(await doesCompileThrow(template, opts)).toBe(false);
     });
 
     it(`should parse ${lang}`, async () => {