-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Fix a crash when a pylint must display unicode raising a UnicodeEncodeError #9732
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
base: main
Are you sure you want to change the base?
Fix a crash when a pylint must display unicode raising a UnicodeEncodeError #9732
Conversation
We should probably aim to keep the |
|
@@ -0,0 +1 @@ | |||
comparison-of-constants:3:7:3:37::"Comparison between constants: 'π == ' has a constant value":HIGH |
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.
comparison-of-constants:3:7:3:37::"Comparison between constants: 'π == ' has a constant value":HIGH | |
comparison-of-constants:3:7:3:37::"Comparison between constants: '"\U00010000" == "\ud800\udc00"' has a constant value":HIGH |
Imo we should aim for this, but I have an intuition that the ast module internal might be involved.
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.
Yes, but I don't think we can. self.out
is utf-8
encoded so we always need to encode/escape to utf-8
.
6c12004
to
96592c9
Compare
π€ Effect of this PR on checked open source code: π€ Effect on astroid:
The following messages are no longer emitted:
Effect on home-assistant:
The following messages are no longer emitted:
Effect on poetry-core:
The following messages are no longer emitted:
Effect on black:
The following messages are no longer emitted:
Effect on music21:
This comment was truncated because GitHub allows only 65536 characters in a comment. This comment was generated for commit 96592c9 |
Type of Changes
Description
Work in progress for #8736, this is probably not the right fix, but it's a fix. Hoping for a surrogates/unicode expert to chime in with the right approach π !