Skip to content

Use reported covered lines - W-16286380 - #1660

Merged
peternhale merged 1 commit into
mainfrom
ph/W-16286380-coverage-reports
Oct 6, 2026
Merged

peternhale merged 1 commit into
mainfrom
ph/W-16286380-coverage-reports

Conversation

@peternhale

Copy link
Copy Markdown
Contributor

Summary

  • use locationsCovered from API v68+ code-coverage responses rather than inferring covered lines
  • retain inferred-line behavior for older API versions or responses without locationsCovered
  • update generated command schemas and coverage tests

Testing

  • yarn test:only -- test/utils/coverage.test.ts
  • pre-push hook: lint, compile, test:only, and JSON-schema validation

W-16286380

@peternhale
peternhale marked this pull request as ready for review October 6, 2026 15:14
@peternhale
peternhale merged commit c7af718 into main Oct 6, 2026
43 of 44 checks passed
@peternhale
peternhale deleted the ph/W-16286380-coverage-reports branch October 6, 2026 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants