Skip to content

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

Merged
merged 7 commits into from
Mar 12, 2020
Merged

Fix 33436 #35225

merged 7 commits into from
Mar 12, 2020

Conversation

jack-williams
Copy link
Collaborator

@jack-williams jack-williams commented Nov 20, 2019

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.

@jack-williams jack-williams mentioned this pull request Nov 20, 2019
@jack-williams
Copy link
Collaborator Author

jack-williams commented Jan 20, 2020

Sorry @DanielRosenwasser, a bad merge flipped the elaboration arguments back. Fixed now.

@sandersn sandersn added the For Backlog Bug PRs that fix a backlog bug label Feb 3, 2020
@sandersn
Copy link
Member

@weswigham the reviewers box is broken, but can you look at this too?

Copy link
Member

@weswigham weswigham left a 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).

@sandersn sandersn merged commit dcc7394 into microsoft:master Mar 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Backlog Bug PRs that fix a backlog bug
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Don't mention constraints in errors when type parameters don't explicitly specify them
5 participants