-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Closed
Feature
Copy link
Labels
breaking changefeat: coverageIssues and PRs related to the coverage featureIssues and PRs related to the coverage featurep3-significantHigh priority enhancement (priority)High priority enhancement (priority)
Milestone
Description
Clear and concise description of the problem
Continuation for:
There should be no need for old non-AST based remapping (v8-to-istanbul) once vitest@3.2.0 has been released. However the coverage diffs between these two remapping methods is quite much, so we should consider this as breaking change. Users are very likely seeing different, more accurate results with the new experimentalAstAwareRemapping option.
Removing the old remapping method simplifies @vitest/coverage-v8 codebase a lot.
Suggested solution
- Remove
v8-to-istanbulcompletely - Remove
experimentalAstAwareRemappingoption and default to AST based remapping viaast-v8-to-istanbulalways
Alternative
No response
Additional context
No response
Validations
- Follow our Code of Conduct
- Read the Contributing Guidelines.
- Read the docs.
- Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
breaking changefeat: coverageIssues and PRs related to the coverage featureIssues and PRs related to the coverage featurep3-significantHigh priority enhancement (priority)High priority enhancement (priority)