-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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 E0403, E0404, E0405, E0407, E0428 and E0433 error explanation #27084
Conversation
// parameter in this type parameter list | ||
``` | ||
|
||
Please verify you didn't mispell the type parameters. Example: |
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.
Please verify that none of the type params are misspelled, and rename any clashing parameters.
(missing "that"; passive voice)
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.
And here we go again haha.
For travis actual issue, we'll have to wait that #27067 is merged. |
} | ||
``` | ||
|
||
Please verify you didn't mispelled the method name and you used the good |
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.
s/mispelled/misspell/
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.
s/the good/the correct
☔ The latest upstream changes (presumably #27098) made this pull request unmergeable. Please resolve the merge conflicts. |
c4669f8
to
4684780
Compare
☔ The latest upstream changes (presumably #27120) made this pull request unmergeable. Please resolve the merge conflicts. |
@bors r+ rollup |
📌 Commit 91cc413 has been approved by |
@bors r- @GuillaumeGomez this phrase is used 3 times: 'please verify you didn't misspelled'. Please change it to 'please verify you didn't misspell'. |
Fixed. |
@bors r+ rollup |
📌 Commit 2e919b4 has been approved by |
Part of #24407.