Skip to content

Bugfix/ignore inp rules for tests dir#757

Merged
nathanjmcdougall merged 3 commits into
mainfrom
bugfix/ignore-inp-rules-for-tests-dir
Jun 18, 2025
Merged

Bugfix/ignore inp rules for tests dir#757
nathanjmcdougall merged 3 commits into
mainfrom
bugfix/ignore-inp-rules-for-tests-dir

Conversation

@nathanjmcdougall

Copy link
Copy Markdown
Collaborator

No description provided.

@nathanjmcdougall nathanjmcdougall linked an issue Jun 18, 2025 that may be closed by this pull request
@codecov

codecov Bot commented Jun 18, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 88.67925% with 6 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/usethis/_tool/rule.py 82.60% 4 Missing ⚠️
src/usethis/_tool/impl/ruff.py 90.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@codspeed-hq

codspeed-hq Bot commented Jun 18, 2025

Copy link
Copy Markdown

CodSpeed Walltime Performance Report

Merging #757 will not alter performance

Comparing bugfix/ignore-inp-rules-for-tests-dir (0223077) with main (59a585c)

Summary

✅ 1 untouched benchmarks

@nathanjmcdougall nathanjmcdougall merged commit 33ae98a into main Jun 18, 2025
18 of 19 checks passed
@nathanjmcdougall nathanjmcdougall deleted the bugfix/ignore-inp-rules-for-tests-dir branch June 18, 2025 05:54
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.

INP rules should not be enabled in the tests dir by default

1 participant