-
Notifications
You must be signed in to change notification settings - Fork 12.9k
Fix 33436 #35225
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
Fix 33436 #35225
Conversation
b0ab212
to
5319858
Compare
tests/baselines/reference/assignmentCompatWithCallSignatures3.errors.txt
Outdated
Show resolved
Hide resolved
5319858
to
69a5e19
Compare
65a409f
to
0d91997
Compare
Sorry @DanielRosenwasser, a bad merge flipped the elaboration arguments back. Fixed now. |
@weswigham the reviewers box is broken, but can you look at this too? |
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.
The new error looks fine to me (assuming @DanielRosenwasser is still fine with this copy), but this is probably going to need a sync with master and re-baseline-update to not break the build on merge (as there's probably newer tests which'll get the new elaboration).
Fixes #33436
Adds new error elaboration when relating to a type parameter and the source is not related to the constraint, or if a constraint does not exist.