-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add better explanation to error message #18665
Conversation
How I fixed it It was easy (I did not expect it to be hard):
That's it. It would be great if others would try to do the same thing when they see an obscure error message. Create an issue first and then fix it according to these steps. These are incremental improvements but they are really valuable in everyday work! |
| | ||
| (Int, Int) => Int | ||
| | ||
| requires a function with 2 parameters. |
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.
Maybe the reader can infer from looking at the printed tree, but perhaps also explain that placeholder parameters always infer parameters 1 level of nesting up from their position (however best to explain that).
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.
Indeed, not sure how to explain that, in a situation like this, where context is missing.
Backports #18665 to the LTS branch. PR submitted by the release tooling. [skip ci]
Fixes #18657