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

Jump to bottom

Implement new upstream rule PT028 #15412

Closed
Zac-HD opened this issue Jan 11, 2025 · 5 comments · Fixed by #15449
Closed

Implement new upstream rule PT028 #15412

Zac-HD opened this issue Jan 11, 2025 · 5 comments · Fixed by #15449
Labels
rule Implementing or modifying a lint rule

Comments

@Zac-HD
Copy link

Zac-HD commented Jan 11, 2025

from m-burst/flake8-pytest-style#324
@MichaReiser MichaReiser added the rule Implementing or modifying a lint rule label Jan 11, 2025
@MichaReiser
Copy link
Member

Thank you

@UnknownPlatypus
Copy link
Contributor

There is also PT029, PT030, and PT031 -> https://github.com/m-burst/flake8-pytest-style?tab=readme-ov-file#change-log

@tjkuson
Copy link
Contributor

tjkuson commented Jan 12, 2025

I'll have a PR up for this tomorrow!

@InSyncWithFoo
Copy link
Contributor

@tjkuson Do you mind if I take this? I too have a branch ready (I completed it without being aware of your comment), but I'm willing to discard it in favor of yours.

@tjkuson
Copy link
Contributor

tjkuson commented Jan 13, 2025

Mine isn't finished so by all means create a PR

@InSyncWithFoo InSyncWithFoo mentioned this issue Jan 13, 2025
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
rule Implementing or modifying a lint rule
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[flake8-pytest-style] Test function parameters with default arguments (PT028) InSyncWithFoo/ruff
5 participants
@MichaReiser @tjkuson @Zac-HD @UnknownPlatypus @InSyncWithFoo