Skip to content

Fix 33436 - #35225

Merged
Nathan Shively-Sanders (sandersn) merged 7 commits into
microsoft:masterfrom
jack-williams:fix33436
Mar 12, 2020
Merged

Fix 33436#35225
Nathan Shively-Sanders (sandersn) merged 7 commits into
microsoft:masterfrom
jack-williams:fix33436

Conversation

@jack-williams

@jack-williamsJack Williams (jack-williams) commented Nov 20, 2019

Copy link
Copy Markdown
Collaborator

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.

Comment threadsrc/compiler/diagnosticMessages.json Outdated
@jack-williams

Jack Williams (jack-williams) commented Jan 20, 2020

Copy link
Copy Markdown
CollaboratorAuthor

Sorry Daniel Rosenwasser (@DanielRosenwasser), a bad merge flipped the elaboration arguments back. Fixed now.

@sandersn

Copy link
Copy Markdown
Member

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

Copy link
Copy Markdown
Member

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 Daniel Rosenwasser (@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
Nathan Shively-Sanders (sandersn) merged commit dcc7394 into microsoft:masterMar 12, 2020
@microsoftMicrosoft (microsoft) locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

For Backlog BugPRs 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

@jack-williams@sandersn@DanielRosenwasser@weswigham@elibarzilay