-
Notifications
You must be signed in to change notification settings - Fork 14.1k
Update the comment in the add_typo_suggestion function #149505
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
|
rustbot has assigned @WaffleLapkin. Use |
|
What is the purpose of this? i.e. is it even important that the comment is traceable to the test? I feel like the diagnostic itself, not the file, is the important part here. |
|
While reading the comments, I became curious about the exact circumstances under which this message was printed, so I looked for the file in question, but couldn't find it. So, I changed the comment to make it easier to find. If you deem it unnecessary, I'll close the PR. |
|
@bors r+ rollup |
Rollup of 6 pull requests Successful merges: - #149393 (expand valid edition range for use-path-segment-kw.rs) - #149427 (Make the capitalization explicit on keyword misspell error) - #149433 (Use a delayed bug for this layout ICE) - #149473 (Tidying up UI tests [7/N]) - #149505 (Update the comment in the add_typo_suggestion function) - #149513 (`rust-analyzer` subtree update) r? `@ghost` `@rustbot` modify labels: rollup
This comment was added to 089810a. However, the test for the comment was moved elsewhere in #145897.