Repository navigation
gh-152490: Fix TypeError when traceback.py formats SyntaxError(NotImplemented) - #152495
Conversation
terryjreedy
left a comment
There was a problem hiding this comment.
I have no opinion on the test change.
|
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
…8qLbF.rst Co-authored-by: Terry Jan Reedy <[email protected]>
Co-authored-by: Terry Jan Reedy <[email protected]>
|
@picnixz i've updated with changes, as per your suggestion. please let me know if we need any further improvements. Thanks :) |
Co-authored-by: Bénédikt Tran <[email protected]>
Co-authored-by: Bénédikt Tran <[email protected]>
Co-authored-by: Bénédikt Tran <[email protected]>
Co-authored-by: Terry Jan Reedy <[email protected]>
terryjreedy
left a comment
There was a problem hiding this comment.
Perhaps a formal approval will clear CI to green. I hope to merge tomorrow (Tuesday) and submit a followup about only replacing blanks with the message.
|
@Aniketsy I have never seen these failures and don't think they should be possible for pure python PR. Did you possibly touch, view, load in editor, or otherwise somehow change last-change date or something of autoconf or generated files in your local repository? (I also asked for help elsewhere from other coredevs.) |
@terryjreedy sorry but i don't think i've touched anything, you mentioned.. and yesterday i thought it was from extra line in whatsnew entry, so i removed that but i guess i was wrong. |
|
lets see if this fixes |
|
@terryjreedy i've fixed 😊 |
traceback.py|
Thanks @Aniketsy for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14, 3.15. |
|
GH-160009 is a backport of this pull request to the 3.15 branch. |
|
GH-160010 is a backport of this pull request to the 3.14 branch. |
…(NotImplemented) (GH-152495) (#160010) gh-152490: Fix TypeError when traceback.py formats SyntaxError(NotImplemented) (GH-152495) Method traceback.TracebackException._format_syntax_error contains line `msg = self.msg or "<no detail available>"`. Since 3.14, bool(NotImplemented) raises TypeError and hence so does `self.msg or ...`. Catch it in a try statement. --------- (cherry picked from commit 5503a5e) Co-authored-by: Aniket <[email protected]> Co-authored-by: Terry Jan Reedy <[email protected]> Co-authored-by: Bénédikt Tran <[email protected]>
…(NotImplemented) (GH-152495) (#160009) gh-152490: Fix TypeError when traceback.py formats SyntaxError(NotImplemented) (GH-152495) Method traceback.TracebackException._format_syntax_error contains line `msg = self.msg or "<no detail available>"`. Since 3.14, bool(NotImplemented) raises TypeError and hence so does `self.msg or ...`. Catch it in a try statement. --------- (cherry picked from commit 5503a5e) Co-authored-by: Aniket <[email protected]> Co-authored-by: Terry Jan Reedy <[email protected]> Co-authored-by: Bénédikt Tran <[email protected]>
Method traceback.TracebackException._format_syntax_error contains line
msg = self.msg or "<no detail available>".Since 3.14, bool(NotImplemented) raises TypeError and hence so does
self.msg or .... Catch it in a try statement.