feat!: enable coverage.ignoreEmptyLines by default (#5543)

vitest-dev · May 7, 2024 · 3199494 · 3199494
1 parent ee097db
commit 3199494
Show file tree

Hide file tree

Showing 7 changed files with 339 additions and 2,109 deletions.
diff --git a/docs/config/index.md b/docs/config/index.md
@@ -1374,7 +1374,7 @@ Sets thresholds for files matching the glob pattern.
 #### coverage.ignoreEmptyLines
 
 - **Type:** `boolean`
-- **Default:** `false`
+- **Default:** `true` (`false` in v1)
 - **Available for providers:** `'v8'`
 - **CLI:** `--coverage.ignoreEmptyLines=<boolean>`
 

diff --git a/packages/vitest/src/defaults.ts b/packages/vitest/src/defaults.ts
@@ -43,7 +43,7 @@ export const coverageConfigDefaults: ResolvedCoverageOptions = {
   reporter: [['text', {}], ['html', {}], ['clover', {}], ['json', {}]],
   extension: ['.js', '.cjs', '.mjs', '.ts', '.mts', '.cts', '.tsx', '.jsx', '.vue', '.svelte', '.marko'],
   allowExternal: false,
-  ignoreEmptyLines: false,
+  ignoreEmptyLines: true,
   processingConcurrency: Math.min(20, os.availableParallelism?.() ?? os.cpus().length),
 }