Removed support for the "--config-precedence" option

Author: fabiospampinato

src/bin.ts

@@ -118,10 +118,6 @@ const makeBin = (): Bin => {
       .option("--config-path <path>", "Path to a Prettier configuration file (.prettierrc, package.json, prettier.config.js)", {
         section: "Config",
       })
-      .option("--config-precedence <cli-override|file-override>", 'Define in which order config files and CLI options should be evaluated.\nDefaults to "cli-override"', {
-        section: "Config",
-        enum: ["cli-override", "file-override"],
-      })
       .option("--no-editorconfig", "Don't take .editorconfig into account when parsing configuration", {
         section: "Config",
         default: true,
@@ -304,8 +300,8 @@ const makeWarnedPluggableBin = async (): Promise<Bin> => {
     exit('The "--find-config-path" is not currently supported, please open an issue on GitHub if you need it');
   }
 
-  if (args["config-precedence"] === "prefer-file") {
-    exit('The "prefer-file" value for "--config-precedence" is not currently supported, please open an issue on GitHub if you need it');
+  if (args["config-precedence"]) {
+    exit('The "config-precedence" option is not currently supported, please open an issue on GitHub if you need it');
   }
 
   if (args["file-info"]) {

src/prettier_serial.ts

@@ -38,12 +38,11 @@ async function format(
   formatOptions = await resolve(formatOptions);
   const pluginsBuiltin = await getPluginsBuiltin();
   const plugins = await getPluginsOrExit(formatOptions.plugins || []);
-  const pluginsOverride = contextOptions.configPrecedence !== "file-override";
 
   const options = {
     ...pluginsDefaultOptions,
-    ...(pluginsOverride ? formatOptions : pluginsCustomOptions),
-    ...(pluginsOverride ? pluginsCustomOptions : formatOptions),
+    ...formatOptions,
+    ...pluginsCustomOptions,
     ...contextOptions,
     filepath: filePath,
     plugins: [...pluginsBuiltin, ...plugins],

src/types.ts

@@ -1,7 +1,6 @@
 type Bin = ReturnType<typeof import("tiny-bin").default>;
 
 type ContextOptions = {
-  configPrecedence?: "cli-override" | "file-override";
   cursorOffset?: number;
   rangeEnd?: number;
   rangeStart?: number;

src/utils.ts

@@ -111,9 +111,9 @@ function getGlobPaths(rootPath: string, globs: string[], withNodeModules: boolea
 }
 
 async function getModule<T = unknown>(modulePath: string): Promise<T> {
-    const moduleExports = await import(url.pathToFileURL(modulePath).href);
-    const module = moduleExports.default || moduleExports.exports || moduleExports;
-    return module;
+  const moduleExports = await import(url.pathToFileURL(modulePath).href);
+  const module = moduleExports.default || moduleExports.exports || moduleExports;
+  return module;
 }
 
 function getModulePath(name: string, rootPath: string): string {
@@ -420,13 +420,6 @@ function normalizeContextOptions(options: unknown): ContextOptions {
 
   const contextOptions: ContextOptions = {};
 
-  if ("configPrecedence" in options) {
-    const value = options.configPrecedence;
-    if (isString(value) && (value === "cli-override" || value === "file-override")) {
-      contextOptions.configPrecedence = value;
-    }
-  }
-
   if ("cursorOffset" in options) {
     const value = Number(options.cursorOffset);
     if (isInteger(value) && value >= 0) {

test/__tests__/__snapshots__/early-exit.js.snap

@@ -83,9 +83,6 @@ exports[`show usage with --help shows help text 1`] = `
     
     --no-config               Do not look for a configuration file
     --config-path <path>      Path to a Prettier configuration file (.prettierrc, package.json, prettier.config.js)
-    --config-precedence <cli-override|file-override>
-                              Define in which order config files and CLI options should be evaluated.
-                              Defaults to "cli-override"
     --no-editorconfig         Don't take .editorconfig into account when parsing configuration
     --no-ignore               Do not look for an ignore file
     --ignore-path <path...>   Path to a file with patterns describing files to ignore