Skip to content

Move tool management methods into Tool to reduce duplication#1268

Merged
nathanjmcdougall merged 2 commits intomainfrom
559-reduce-duplication-in-rule-management-methods-for-tools
Dec 31, 2025
Merged

Move tool management methods into Tool to reduce duplication#1268
nathanjmcdougall merged 2 commits intomainfrom
559-reduce-duplication-in-rule-management-methods-for-tools

Conversation

@nathanjmcdougall
Copy link
Copy Markdown
Collaborator

Duplication was between RuffTool and DeptryTool

@nathanjmcdougall nathanjmcdougall linked an issue Dec 31, 2025 that may be closed by this pull request
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 31, 2025

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/usethis/_tool/impl/deptry.py 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Dec 31, 2025

CodSpeed Performance Report

Merging #1268 will not alter performance

Comparing 559-reduce-duplication-in-rule-management-methods-for-tools (0071b24) with main (e676872)

Summary

✅ 2 untouched

@nathanjmcdougall nathanjmcdougall merged commit 313976c into main Dec 31, 2025
18 of 19 checks passed
@nathanjmcdougall nathanjmcdougall deleted the 559-reduce-duplication-in-rule-management-methods-for-tools branch December 31, 2025 01:17
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.

Reduce duplication in rule management methods for tools

1 participant