Skip to content

PYTHON-3900 Add a check for PR diff coverage#2943

Open
blink1073 wants to merge 1 commit into
mongodb:mainfrom
blink1073:PYTHON-3900
Open

PYTHON-3900 Add a check for PR diff coverage#2943
blink1073 wants to merge 1 commit into
mongodb:mainfrom
blink1073:PYTHON-3900

Conversation

@blink1073

@blink1073 blink1073 commented Jul 20, 2026

Copy link
Copy Markdown
Member

PYTHON-3900

Changes in this PR

Enable Codecov's patch coverage status check so pull requests are gated on test coverage of newly added/modified lines (80% target).

Test Plan

Validated the updated .codecov.yml against Codecov's config validation endpoint (returned "Valid!").

Checklist

Checklist for Author

  • Did you update the changelog (if necessary)?
  • Is there test coverage?
  • Is any followup work tracked in a JIRA ticket? If so, add link(s). PYTHON-5949 to track follow up work to add GitHub status checks.

Checklist for Reviewer

  • Does the title of the PR reference a JIRA Ticket?
  • Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
  • Is all relevant documentation (README or docstring) updated?

@blink1073
blink1073 marked this pull request as ready for review July 21, 2026 00:08
@blink1073
blink1073 requested a review from a team as a code owner July 21, 2026 00:08
@blink1073
blink1073 requested review from aclark4life and Copilot July 21, 2026 00:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repository’s Codecov configuration to enforce patch (diff) coverage targets on pull requests, helping ensure newly added/modified lines are adequately covered by tests.

Changes:

  • Enables Codecov patch coverage status reporting.
  • Sets a patch coverage target of 80% with 0% threshold (no allowed drop below target).

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