-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Add 9 more error explanations, improve some error messages. #25688
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
Conversation
r? @pcwalton (rust_highfive has picked a reviewer for you, use r? to override) |
Of the new error codes (E0380-E0394), I've managed to trigger every error except E0385, E0388, and E0389. By the way, it seems like E0385 might not be possible to trigger, because the
|
5a7da25
to
412e549
Compare
☔ The latest upstream changes (presumably #25593) made this pull request unmergeable. Please resolve the merge conflicts. |
Also adds explanations for E0380 and E0381.
Rebased. |
@bors: r+ |
📌 Commit 570a043 has been approved by |
- Adds explanations for E0055, E0089, E0192, E0261-E0263, E0318 - Improves explanations for E0250, E0368, E0372. - Converts 15 diagnostics to have error codes (E0380-E0394). Adds an explanation for E0380. - The E0087-E0090 messages currently look like "expected {} parameter(s) found {} parameter(s)". This changes them to either use "parameter" or "parameters", based on the number. This is, in part, more progress towards #24407
This is, in part, more progress towards #24407