Recognise magic methods #81
Conversation
|
I see that the linting check failed, but that seems to because of the job setup rather than because of my code (?) If you can let me know how to fix it I'd be happy to adjust as needed. |
|
Yep lining is broken but its not your fault :) I'll fix it on main, I'll let you know when to rebase |
| docstring | ||
| elif | ||
| endswith | ||
| enum | ||
| env | ||
| environ | ||
| eq |
There was a problem hiding this comment.
Ideally we would only allow theses exclusively in the context of __ __. But right now there is no way to do that in flake8-spellcheck
Something for me to consider in the future :)
There was a problem hiding this comment.
Yes - did think of that, especially the 2-letter names
I did some investigation, but not obvious how to achieve that
|
@jalanb please rebase on latest |
|
Rebased from upstream's main, lints failed on All tests now passing |
Pull upstream changes
Test a Python class with all possible dunder methods
These are valid names for magic methods
But are already recognised as English (en_US)
Fixes #80
Adds the missing names to
python.txtAdds a test case to check a Python
classwhich includes every possible magic method name