Remove deprecated markdownlint-cli2-config and markdownlint-cli2-fix entry points for command-line, Docker, and pre-commit.

Author: DavidAnson

.gitattributes

@@ -1,3 +1 @@
 markdownlint-cli2.js eol=lf
-markdownlint-cli2-config.js eol=lf
-markdownlint-cli2-fix.js eol=lf

.pre-commit-hooks.yaml

@@ -5,59 +5,17 @@
   language: node
   types: [markdown]
   minimum_pre_commit_version: 0.15.0
-- id: markdownlint-cli2-config
-  name: markdownlint-cli2-config
-  description: "Checks the style of Markdown/CommonMark files with a non-root configuration."
-  entry: markdownlint-cli2-config
-  language: node
-  types: [markdown]
-  minimum_pre_commit_version: 0.15.0
-- id: markdownlint-cli2-fix
-  name: markdownlint-cli2-fix
-  description: "Checks and fixes the style of Markdown/CommonMark files."
-  entry: markdownlint-cli2-fix
-  language: node
-  types: [markdown]
-  minimum_pre_commit_version: 0.15.0
 - id: markdownlint-cli2-docker
   name: markdownlint-cli2-docker
   description: "Checks the style of Markdown/CommonMark files."
   entry: davidanson/markdownlint-cli2
   language: docker_image
   types: [markdown]
   minimum_pre_commit_version: 0.15.0
-- id: markdownlint-cli2-config-docker
-  name: markdownlint-cli2-config-docker
-  description: "Checks the style of Markdown/CommonMark files with a non-root configuration."
-  entry: davidanson/markdownlint-cli2 markdownlint-cli2-config
-  language: docker_image
-  types: [markdown]
-  minimum_pre_commit_version: 0.15.0
-- id: markdownlint-cli2-fix-docker
-  name: markdownlint-cli2-fix-docker
-  description: "Checks and fixes the style of Markdown/CommonMark files."
-  entry: davidanson/markdownlint-cli2 markdownlint-cli2-fix
-  language: docker_image
-  types: [markdown]
-  minimum_pre_commit_version: 0.15.0
 - id: markdownlint-cli2-rules-docker
   name: markdownlint-cli2-rules-docker
   description: "Checks the style of Markdown/CommonMark files."
   entry: davidanson/markdownlint-cli2-rules
   language: docker_image
   types: [markdown]
   minimum_pre_commit_version: 0.15.0
-- id: markdownlint-cli2-config-rules-docker
-  name: markdownlint-cli2-config-rules-docker
-  description: "Checks the style of Markdown/CommonMark files with a non-root configuration."
-  entry: davidanson/markdownlint-cli2-rules markdownlint-cli2-config
-  language: docker_image
-  types: [markdown]
-  minimum_pre_commit_version: 0.15.0
-- id: markdownlint-cli2-fix-rules-docker
-  name: markdownlint-cli2-fix-rules-docker
-  description: "Checks and fixes the style of Markdown/CommonMark files."
-  entry: davidanson/markdownlint-cli2-rules markdownlint-cli2-fix
-  language: docker_image
-  types: [markdown]
-  minimum_pre_commit_version: 0.15.0

README.md

@@ -128,17 +128,6 @@ below) to `true` (though it can still be overridden by a configuration file):
 markdownlint-cli2 --fix "**/*.md" "#node_modules"
 ```
 
-> **Deprecated**
->
-> The following command behaves similarly, defaulting the `fix` property to
-> `true`:
->
-> ```bash
-> markdownlint-cli2-fix "**/*.md" "#node_modules"
-> ```
->
-> Otherwise, `markdownlint-cli2-fix` behaves the same as `markdownlint-cli2`.
-
 In cases where it is not convenient to store a configuration file in the root
 of a project, the `--config` argument can be used to provide a path to any
 supported configuration file (except `package.json`):
@@ -152,17 +141,6 @@ above. For example, `.markdownlint.json` or `example.markdownlint-cli2.jsonc`.
 The specified configuration file will be loaded, parsed, and applied as a base
 configuration for the current directory - which will then be handled normally.
 
-> **Deprecated**
->
-> The following command behaves similarly, accepting a base configuration file
-> path as its first parameter:
->
-> ```bash
-> markdownlint-cli2-config "config/.markdownlint-cli2.jsonc" "**/*.md" "#node_modules"
-> ```
->
-> Otherwise, `markdownlint-cli2-config` behaves the same as `markdownlint-cli2`.
-
 ### Container Image
 
 A container image [`davidanson/markdownlint-cli2`][docker-hub-markdownlint-cli2]
@@ -189,15 +167,6 @@ Notes:
     docker run -w /myfolder -v $PWD:/myfolder davidanson/markdownlint-cli2:v0.11.0 "**/*.md" "#node_modules"
     ```
 
-> **Deprecated**
->
-> To invoke the `markdownlint-cli2-config` or `markdownlint-cli2-fix` commands
-> instead, use Docker's `--entrypoint` flag:
->
-> ```bash
-> docker run -v $PWD:/workdir --entrypoint="markdownlint-cli2-fix" davidanson/markdownlint-cli2:v0.11.0 "**/*.md" "#node_modules"
-> ```
-
 For convenience, the container image
 [`davidanson/markdownlint-cli2-rules`][docker-hub-markdownlint-cli2-rules]
 includes the latest versions of custom rules published to npm with the tag

markdownlint-cli2-config.js

@@ -896,8 +896,7 @@ const main = async (params) => {
     optionsOverride,
     fileContents,
     nonFileContents,
-    noRequire,
-    name
+    noRequire
   } = params;
   let {
     noGlobs
@@ -911,7 +910,7 @@ const main = async (params) => {
   const baseDir = posixPath(baseDirSystem);
   // Output banner
   logMessage(
-    `${name || packageName} v${packageVersion} (${libraryName} v${libraryVersion})`
+    `${packageName} v${packageVersion} (${libraryName} v${libraryVersion})`
   );
   // Merge and process args/argv
   let fixDefault = false;

markdownlint-cli2-fix.js

@@ -15,9 +15,7 @@
     "./markdownlint/helpers": "./export-markdownlint-helpers.js"
   },
   "bin": {
-    "markdownlint-cli2": "markdownlint-cli2.js",
-    "markdownlint-cli2-config": "markdownlint-cli2-config.js",
-    "markdownlint-cli2-fix": "markdownlint-cli2-fix.js"
+    "markdownlint-cli2": "markdownlint-cli2.js"
   },
   "homepage": "https://github.com/DavidAnson/markdownlint-cli2",
   "repository": {
@@ -56,8 +54,6 @@
     "export-markdownlint-helpers.js",
     "LICENSE",
     "markdownlint-cli2.js",
-    "markdownlint-cli2-config.js",
-    "markdownlint-cli2-fix.js",
     "merge-options.js",
     "README.md",
     "resolve-and-require.js",

markdownlint-cli2.js

@@ -159,30 +159,6 @@ const testCases = ({
     "cwd": "no-config"
   });
 
-  testCase({
-    "name": "no-arguments-fix",
-    "script": "markdownlint-cli2-fix.js",
-    "args": [],
-    "exitCode": 2,
-    "cwd": "no-config"
-  });
-
-  testCase({
-    "name": "no-arguments-config",
-    "script": "markdownlint-cli2-config.js",
-    "args": [],
-    "exitCode": 2,
-    "cwd": "no-config"
-  });
-
-  testCase({
-    "name": "one-argument-config",
-    "script": "markdownlint-cli2-config.js",
-    "args": [ "../config-files/cfg/.markdownlint-cli2.jsonc" ],
-    "exitCode": 2,
-    "cwd": "no-config"
-  });
-
   testCase({
     "name": "no-arguments-config-arg",
     "args": [ "--config" ],
@@ -631,23 +607,6 @@ const testCases = ({
     "post": deleteDirectory
   });
 
-  testCase({
-    "name": "fix-default-true",
-    "script": "markdownlint-cli2-fix.js",
-    "args": [ "**/*.md" ],
-    "exitCode": 1,
-    "cwd": directoryName("fix-default-true"),
-    "pre": copyDirectory,
-    "post": deleteDirectory
-  });
-
-  testCase({
-    "name": "fix-default-true-override",
-    "script": "markdownlint-cli2-fix.js",
-    "args": [ "**/*.md" ],
-    "exitCode": 1
-  });
-
   testCase({
     "name": "fix-default-true-arg",
     "shadow": "fix-default-true",
@@ -679,33 +638,6 @@ const testCases = ({
   ];
   for (const configFile of configFiles) {
     const usesRequire = isModule(configFile);
-    testCase({
-      "name": `config-files-${configFile}`,
-      "script": "markdownlint-cli2-config.js",
-      "args": [ `cfg/${configFile}`, "**/*.md" ],
-      "exitCode": 1,
-      "cwd": "config-files",
-      usesRequire
-    });
-    testCase({
-      "name": `config-files-${configFile}-alternate`,
-      "script": "markdownlint-cli2-config.js",
-      "args": [ `cfg/alternate${configFile}`, "**/*.md" ],
-      "exitCode": 1,
-      "cwd": "config-files",
-      usesRequire
-    });
-    testCase({
-      "name": `config-files-${configFile}-absolute`,
-      "script": "markdownlint-cli2-config.js",
-      "args": [
-        path.join(__dirname, "config-files", `cfg/${configFile}`),
-        "**/*.md"
-      ],
-      "exitCode": 1,
-      "cwd": "config-files",
-      usesRequire
-    });
     testCase({
       "name": `config-files-${configFile}-arg`,
       "args": [ "--config", `cfg/${configFile}`, "**/*.md" ],
@@ -748,15 +680,6 @@ const testCases = ({
   ];
   for (const [ invalidConfigFile, stderrRe ] of invalidConfigFiles) {
     const usesRequire = isModule(invalidConfigFile);
-    testCase({
-      "name": `config-files-${invalidConfigFile}-invalid`,
-      "script": "markdownlint-cli2-config.js",
-      "args": [ `cfg/${invalidConfigFile}`, "**/*.md" ],
-      "exitCode": 2,
-      stderrRe,
-      "cwd": "config-files",
-      usesRequire
-    });
     testCase({
       "name": `config-files-${invalidConfigFile}-invalid-arg`,
       "args": [ "--config", `cfg/${invalidConfigFile}`, "**/*.md" ],
@@ -774,14 +697,6 @@ const testCases = ({
   ];
   for (const redundantConfigFile of redundantConfigFiles) {
     const usesRequire = isModule(redundantConfigFile);
-    testCase({
-      "name": `config-files-${redundantConfigFile}-redundant`,
-      "script": "markdownlint-cli2-config.js",
-      "args": [ redundantConfigFile, "*.md" ],
-      "exitCode": 1,
-      "cwd": redundantConfigFile.slice(1).replace(".", "-"),
-      usesRequire
-    });
     testCase({
       "name": `config-files-${redundantConfigFile}-redundant-arg`,
       "args": [ "--config", redundantConfigFile, "*.md" ],
@@ -791,41 +706,6 @@ const testCases = ({
     });
   }
 
-  testCase({
-    "name": "config-file-unrecognized",
-    "script": "markdownlint-cli2-config.js",
-    "args": [ "cfg/unrecognized.jsonc", "**/*.md" ],
-    "exitCode": 2,
-    "stderrRe":
-      /Configuration file "[^"]*cfg\/unrecognized\.jsonc" is unrecognized; its name should be \(or end with\) one of the supported types \(e\.g\., "\.markdownlint\.json" or "example\.markdownlint-cli2\.jsonc"\)\./u,
-    "cwd": "config-files"
-  });
-
-  testCase({
-    "name": "config-relative-commonjs",
-    "script": "markdownlint-cli2-config.js",
-    "args": [ "config/.markdownlint-cli2.jsonc", "viewme.md", "link.md" ],
-    "exitCode": 1,
-    "usesRequire": true
-  });
-
-  testCase({
-    "name": "config-relative-module",
-    "script": "markdownlint-cli2-config.js",
-    "args": [ "config/.markdownlint-cli2.jsonc", "viewme.md", "link.md" ],
-    "exitCode": 1
-  });
-
-  testCase({
-    "name": "config-with-fix",
-    "script": "markdownlint-cli2-config.js",
-    "args": [ "config/.markdownlint-cli2.jsonc", "viewme.md", "info.md" ],
-    "exitCode": 0,
-    "cwd": directoryName("config-with-fix"),
-    "pre": copyDirectory,
-    "post": deleteDirectory
-  });
-
   testCase({
     "name": "config-file-unrecognized-arg",
     "args": [ "--config", "cfg/unrecognized.jsonc", "**/*.md" ],