Skip to content
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

[pylint] Do not wrap function calls in parentheses in the fix for unnecessary-dunder-call (PLC2801) #14601

Merged
merged 3 commits into from
Nov 26, 2024

Conversation

dylwil3
Copy link
Collaborator

@dylwil3 dylwil3 commented Nov 26, 2024

In the fix for unnecessary-dunder-call (PLC2801), when replacing the dunder call for a builtin by the function call, we no longer wrap the function call in parentheses.

Closes #14597

Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@dylwil3 dylwil3 changed the title [pylint] Do not wrap function calls in parentheses in the fix for dict-index-missing-items (PLC0206) [pylint] Do not wrap function calls in parentheses in the fix for unnecessary-dunder-call (PLC2801) Nov 26, 2024
@MichaReiser MichaReiser added bug Something isn't working fixes Related to suggested fixes for violations labels Nov 26, 2024
@dylwil3 dylwil3 merged commit 24c90d6 into astral-sh:main Nov 26, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixes Related to suggested fixes for violations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unnecessary parentheses when fix PLC2801
2 participants