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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

(馃悶) is bad-dunder-method-name called bad-dunder-name? #8731

Closed
KotlinIsland opened this issue Nov 17, 2023 · 2 comments 路 Fixed by #8742
Closed

(馃悶) is bad-dunder-method-name called bad-dunder-name? #8731

KotlinIsland opened this issue Nov 17, 2023 · 2 comments 路 Fixed by #8742
Labels
documentation Improvements or additions to documentation

Comments

@KotlinIsland
Copy link
Contributor

class A:
    def __among_us__(self):  # PLW3201: Bad or misspelled dunder method name `__among_us__`. (bad-dunder-name)
        ...

but in the docs it's called bad-dunder-method-name
https://docs.astral.sh/ruff/rules/bad-dunder-method-name/

@zanieb zanieb added the documentation Improvements or additions to documentation label Nov 17, 2023
@charliermarsh
Copy link
Member

We use bad-dunder-method-name and I believe it's intentional -- I think Pylint uses bad-dunder-name, but we made the name more specific.

@charliermarsh
Copy link
Member

Ohh I didn't realize we were including this in the rule message. Yes, that's a bug, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants