-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
gh-134036: Improve error messages for invalid raise
statements
#134077
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
Follow Lee on X/Twitter - Father, Husband, Serial builder creating AI, crypto, games & web tools. We are friends :) AI Will Come To Life!
Check out: eBank.nz (Art Generator) |
Netwrck.com (AI Tools) |
Text-Generator.io (AI API) |
BitBank.nz (Crypto AI) |
ReadingTime (Kids Reading) |
RewordGame |
BigMultiplayerChess |
WebFiddle |
How.nz |
Helix AI Assistant
Conversation
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.
LGTM! It's simple enough that shouldn't create any problems and it is a clear improvement.
Great improvement! ❤️ |
I think this passed pre-merge CI but failed post-merge because rebasing it is what broke it... We can roll this back or it can be fixed-forward. If this can't be fixed quickly it should be rolled back. (/cc @hugovk.) |
I wrote #135204 to update test_syntax, the fix is trivial (update error messages). |
Better error message wordings are welcome!
raise from ValueError()
error message #134036