Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add PT028 (checks for default arguments in test functions) #324

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

m-burst
Copy link
Owner

@m-burst m-burst commented Jan 10, 2025

Fixes #316

Copy link

codecov bot commented Jan 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.76%. Comparing base (a7fee05) to head (92c0803).
Report is 26 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #324      +/-   ##
==========================================
+ Coverage   96.66%   96.76%   +0.10%     
==========================================
  Files          13       14       +1     
  Lines         599      619      +20     
  Branches      129      126       -3     
==========================================
+ Hits          579      599      +20     
  Misses          7        7              
  Partials       13       13              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@m-burst m-burst marked this pull request as ready for review January 10, 2025 14:20
@m-burst m-burst merged commit c894530 into master Jan 10, 2025
7 checks passed
@m-burst m-burst deleted the issue-316 branch January 10, 2025 14:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New rule: warn if the function definition for a test includes default arguments
1 participant