Ignore node_modules/ by default

README.md

@@ -30,10 +30,10 @@ markdownlint --help
     -c, --config [configFile]                   configuration file (JSON, JSONC, JS, YAML, or TOML)
     -d, --dot                                   include files/folders with a dot (for example `.github`)
     -f, --fix                                   fix basic errors (does not work with STDIN)
-    -i, --ignore [file|directory|glob]          file(s) to ignore/exclude (default: [])
+    -i, --ignore [file|directory|glob]          file(s) to ignore/exclude (default: ['node_modules'])
+    -p, --ignore-path [file]                    path to file with ignore pattern(s)
     -j, --json                                  write issues in json format
     -o, --output [outputFile]                   write issues to file (no console)
-    -p, --ignore-path [file]                    path to file with ignore pattern(s)
     -q, --quiet                                 do not write issues to STDOUT
     -r, --rules  [file|directory|glob|package]  include custom rule files (default: [])
     -s, --stdin                                 read from STDIN (does not work with files)

markdownlint.js

@@ -199,10 +199,10 @@ program
   .option('-c, --config [configFile]', 'configuration file (JSON, JSONC, JS, or YAML)')
   .option('-d, --dot', 'include files/folders with a dot (for example `.github`)')
   .option('-f, --fix', 'fix basic errors (does not work with STDIN)')
-  .option('-i, --ignore [file|directory|glob]', 'file(s) to ignore/exclude', concatArray, [])
+  .option('-i, --ignore [file|directory|glob]', 'file(s) to ignore/exclude', concatArray, ['node_modules'])
+  .option('-p, --ignore-path [file]', 'path to file with ignore pattern(s)')
   .option('-j, --json', 'write issues in json format')
   .option('-o, --output [outputFile]', 'write issues to file (no console)')
-  .option('-p, --ignore-path [file]', 'path to file with ignore pattern(s)')
   .option('-q, --quiet', 'do not write issues to STDOUT')
   .option('-r, --rules  [file|directory|glob|package]', 'include custom rule files', concatArray, [])
   .option('-s, --stdin', 'read from STDIN (does not work with files)')

test/default-ignore/incorrect.md

@@ -0,0 +1 @@
+# header

test/test.js

@@ -163,7 +163,7 @@ test('glob linting works with failing files', async t => {
     t.fail();
   } catch (error) {
     t.is(error.stdout, '');
-    t.is(error.stderr.match(errorPattern).length, 15);
+    t.is(error.stderr.match(errorPattern).length, 23);
     t.is(error.exitCode, 1);
   }
 });
@@ -217,7 +217,7 @@ test('glob linting with failing files has fewer errors when ignored by dir', asy
     t.fail();
   } catch (error) {
     t.is(error.stdout, '');
-    t.is(error.stderr.match(errorPattern).length, 8);
+    t.is(error.stderr.match(errorPattern).length, 16);
     t.is(error.exitCode, 1);
   }
 });
@@ -228,7 +228,7 @@ test('glob linting with failing files has fewer errors when ignored by dir and a
     t.fail();
   } catch (error) {
     t.is(error.stdout, '');
-    t.is(error.stderr.match(errorPattern).length, 8);
+    t.is(error.stderr.match(errorPattern).length, 16);
     t.is(error.exitCode, 1);
   }
 });
@@ -413,7 +413,7 @@ test('linting using a toml configuration file works', async t => {
     t.fail();
   } catch (error) {
     t.is(error.stdout, '');
-    t.is(error.stderr.match(errorPattern).length, 15);
+    t.is(error.stderr.match(errorPattern).length, 23);
     t.is(error.exitCode, 1);
   }
 });
@@ -424,7 +424,7 @@ test('linting using a yaml configuration file works', async t => {
     t.fail();
   } catch (error) {
     t.is(error.stdout, '');
-    t.is(error.stderr.match(errorPattern).length, 15);
+    t.is(error.stderr.match(errorPattern).length, 23);
     t.is(error.exitCode, 1);
   }
 });
@@ -665,6 +665,21 @@ test('fixing errors with a glob yields fewer errors', async t => {
   }
 });
 
+test('default ignore is applied correctly', async t => {
+  try {
+    await execa(path.resolve('..', 'markdownlint.js'), ['.'], {
+      cwd: path.join(__dirname, 'default-ignore'),
+      stripFinalNewline: false
+    });
+    t.fail();
+  } catch (error) {
+    const expected = ['incorrect.md:1:8 MD047/single-trailing-newline Files should end with a single newline character', ''].join('\n');
+    t.is(error.stdout, '');
+    t.is(error.stderr.replaceAll('\\', '/'), expected);
+    t.is(error.exitCode, 1);
+  }
+});
+
 test('.markdownlintignore is applied correctly', async t => {
   try {
     await execa(path.resolve('..', 'markdownlint.js'), ['.'], {