Skip to content

Conversation

@nsajko
Copy link
Member

@nsajko nsajko commented Jan 4, 2025

No description provided.

@nsajko nsajko added error handling Handling of exceptions by Julia or the user bugfix This change fixes an existing bug error messages Better, more actionable error messages macros @macros backport 1.10 Change should be backported to the 1.10 release backport 1.11 Change should be backported to release-1.11 labels Jan 4, 2025
@nsajko
Copy link
Member Author

nsajko commented Jan 4, 2025

The current behavior is for @constprop :unknown f() = 3 to fail with a MethodError because of a typo when trying to construct the ArgumentError.

@nsajko nsajko force-pushed the fix_constprop_error_handling branch from 2ab15ed to 30818e6 Compare January 5, 2025 11:15
@aviatesk aviatesk merged commit a3f336f into JuliaLang:master Jan 6, 2025
5 of 7 checks passed
@nsajko nsajko deleted the fix_constprop_error_handling branch January 6, 2025 09:27
This was referenced Jan 28, 2025
nsajko added a commit to nsajko/julia that referenced this pull request Feb 9, 2025
nsajko added a commit to nsajko/julia that referenced this pull request Feb 9, 2025
aviatesk pushed a commit that referenced this pull request Feb 10, 2025
…rror message (#57321)

Backport of PR #56946 to v1.11.

Co-authored-by: Shuhei Kadowaki
<[email protected]>
(cherry picked from commit a3f336f)
aviatesk pushed a commit that referenced this pull request Feb 10, 2025
…rror message (#57320)

Backport of PR #56946 to v1.10.

Co-authored-by: Shuhei Kadowaki
<[email protected]>
(cherry picked from commit a3f336f)
@nsajko nsajko removed backport 1.10 Change should be backported to the 1.10 release backport 1.11 Change should be backported to release-1.11 labels Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This change fixes an existing bug error handling Handling of exceptions by Julia or the user error messages Better, more actionable error messages macros @macros

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants