-
Notifications
You must be signed in to change notification settings - Fork 4
Rename coverage to Coverage.py #631
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename coverage to Coverage.py #631
Conversation
We're not changing the CLI interface from `usethis tool coverage` but everything else needs to change.
CodSpeed Performance ReportMerging #631 will not alter performanceComparing Summary
|
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## main #631 +/- ##
=======================================
Coverage 97.55% 97.55%
=======================================
Files 102 102
Lines 4453 4454 +1
=======================================
+ Hits 4344 4345 +1
Misses 109 109 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
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 renames the internal CoverageTool
class and associated calls to CoveragePyTool
and updates help messages and test expectations to reference "Coverage.py" while keeping the CLI command as coverage
.
- Renamed
CoverageTool
→CoveragePyTool
(class, imports, and function calls) - Updated
name
property and help outputs to use "Coverage.py" - Adjusted tests to reference the new class and display name
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
tests/usethis/_tool/impl/test_coverage.py | Updated import and calls to use CoveragePyTool |
tests/usethis/_interface/test_tool.py | Changed help text and command‐matching logic for Coverage.py |
tests/usethis/_core/test_list.py | Updated expected tool name to "Coverage.py" |
tests/usethis/_core/test_core_tool.py | Switched use_coverage → use_coverage_py and updated output |
src/usethis/_tool/impl/pytest.py | Replaced CoverageTool references with CoveragePyTool |
src/usethis/_tool/impl/pyproject_toml.py | Swapped import of CoverageTool for CoveragePyTool |
src/usethis/_tool/impl/coverage.py | Renamed class and updated print_how_to_use messages |
src/usethis/tool/all.py | Replaced CoverageTool with CoveragePyTool in tool list |
src/usethis/_interface/tool.py | Renamed internal function use and help text to Coverage.py |
src/usethis/_core/tool.py | Changed use_coverage → use_coverage_py and imports |
Comments suppressed due to low confidence (2)
tests/usethis/_tool/impl/test_coverage.py:9
- [nitpick] Rename the test class to
TestCoveragePyTool
to match the renamedCoveragePyTool
class for consistency.
class TestCoverageTool:
tests/usethis/_tool/impl/test_coverage.py:1
- [nitpick] Consider renaming this test file to
test_coverage_py.py
to align with the class under test.
tests/usethis/_tool/impl/test_coverage.py
The `usethis tool coverage` interface is included for backward compatibility.
We're not changing the CLI interface from
usethis tool coverage
but everything else needs to change.