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
Add redundant-u-string-prefix
checker
#4804
Conversation
This adds a checker for u-prefixes for strings, as used in Python 2. Closes pylint-dev#4102
@@ -89,10 +89,14 @@ Release date: TBA | |||
|
|||
Closes #626 | |||
|
|||
* Add ``disable-next`` option: allows using `# pylint: disable-next=msgid` to disable a message for the following line | |||
* Added ``disable-next`` option: allows using `# pylint: disable-next=msgid` to disable a message for the following line |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All other changes also use past tense
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Neat clean feature, thank you !
Could a maintainer force re-run the checks? Something went wrong with the runners I think.. |
I relaunched, CodeQL failed again but it's not a mandatory check. The pypy one seems genuine. |
Oops, looks like I forgot to run the test suite locally. I think I didn't expect pylint to use u-prefixes.. 😅 |
Turns out |
I don't know if it would be "easy" or "elegant", but it would be a change in astroid. Maybe we can do this check only in python 3.8+ ? |
Sure! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great new checker !
doc/whatsnew/<current release.rst>
.Type of Changes
Description
This adds a checker for u-prefixes for strings, as used in Python 2.
Closes #4102