Skip to content

Comments

Fix missing files parameter in print_coverage_table() docstring#4106

Merged
TaoChenOSU merged 2 commits intotaochen/python-add-more-test-coverage-gatefrom
copilot/sub-pr-4104
Feb 19, 2026
Merged

Fix missing files parameter in print_coverage_table() docstring#4106
TaoChenOSU merged 2 commits intotaochen/python-add-more-test-coverage-gatefrom
copilot/sub-pr-4104

Conversation

Copy link
Contributor

Copilot AI commented Feb 19, 2026

The files parameter was added to print_coverage_table() signature but never documented in the docstring's Args section, leaving it undiscoverable from the docs alone.

Description

  • Added files entry to the Args section of print_coverage_table() docstring, describing it as a dictionary of file path to PackageCoverage data used for per-file enforcement.

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
Copilot AI changed the title [WIP] Update unit test coverage gates based on feedback Fix missing files parameter in print_coverage_table() docstring Feb 19, 2026
Copilot AI requested a review from TaoChenOSU February 19, 2026 22:14
@TaoChenOSU TaoChenOSU marked this pull request as ready for review February 19, 2026 22:40
@TaoChenOSU TaoChenOSU merged commit 281ea24 into taochen/python-add-more-test-coverage-gate Feb 19, 2026
2 checks passed
github-merge-queue bot pushed a commit that referenced this pull request Feb 19, 2026
* Add more unit test coverage gates

* Fix missing `files` parameter in `print_coverage_table()` docstring (#4106)

* Initial plan

* Update print_coverage_table docstring to document files parameter

Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
@crickman crickman deleted the copilot/sub-pr-4104 branch February 22, 2026 19:08
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