feat: fmt and lint respect .gitignore file (#26897)

Closes https://github.com/denoland/deno/issues/26573

cli/tools/fmt.rs

@@ -228,6 +228,7 @@ fn collect_fmt_files(
   })
   .ignore_git_folder()
   .ignore_node_modules()
+  .use_gitignore()
   .set_vendor_folder(cli_options.vendor_dir_path().map(ToOwned::to_owned))
   .collect_file_patterns(&deno_config::fs::RealDenoConfigFs, files)
 }

cli/tools/lint/mod.rs

@@ -436,6 +436,7 @@ fn collect_lint_files(
   })
   .ignore_git_folder()
   .ignore_node_modules()
+  .use_gitignore()
   .set_vendor_folder(cli_options.vendor_dir_path().map(ToOwned::to_owned))
   .collect_file_patterns(&deno_config::fs::RealDenoConfigFs, files)
 }

tests/specs/fmt/gitignore/.gitignore

@@ -0,0 +1 @@
+/dist/

tests/specs/fmt/gitignore/__test__.jsonc

@@ -0,0 +1,6 @@
+{
+  "tempDir": true,
+  "args": "fmt --check",
+  "output": "expected.out",
+  "exitCode": 1
+}

tests/specs/fmt/gitignore/dist/file1.js

@@ -0,0 +1,6 @@
+// This file is in `.gitignore` simulating that it's generated by a build tool
+// and should not be linted
+function foo(   )   {
+    console.log(  "hello")
+       }
+   

tests/specs/fmt/gitignore/expected.out

@@ -0,0 +1,10 @@
+
+from [WILDCARD]file2.ts:
+1 | -function foo(   ): any   {
+1 | +function foo(): any {
+2 | - console.log(  "hello")
+2 | +  console.log("hello");
+3 | -    }
+3 | +}
+
+error: Found 1 not formatted file in 1 file

tests/specs/fmt/gitignore/file2.ts

@@ -0,0 +1,3 @@
+function foo(   ): any   {
+ console.log(  "hello")
+    }

tests/specs/lint/gitignore/.gitignore

@@ -0,0 +1 @@
+/dist/

tests/specs/lint/gitignore/__test__.jsonc

@@ -0,0 +1,5 @@
+{
+  "args": "lint",
+  "output": "expected.out",
+  "exitCode": 1
+}

tests/specs/lint/gitignore/dist/file1.js

@@ -0,0 +1,3 @@
+// This file is in `.gitignore` simulating that it's generated by a build tool
+// and should not be linted
+while (false) {}

tests/specs/lint/gitignore/expected.out

@@ -0,0 +1,12 @@
+error[no-empty]: Empty block statement
+ --> [WILDCARD]file2.ts:3:14
+  | 
+3 | } catch (_e) {}
+  |              ^^
+  = hint: Add code or comment to the empty block
+
+  docs: https://lint.deno.land/rules/no-empty
+
+
+Found 1 problem
+Checked 1 file

tests/specs/lint/gitignore/file2.ts

@@ -0,0 +1,6 @@
+try {
+  await Deno.open("./some/file.txt");
+} catch (_e) {}
+
+// deno-lint-ignore no-explicit-any
+function _foo(): any {}