-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Fix some typos #117911
Fix some typos #117911
Conversation
Signed-off-by: cui fliter <[email protected]>
r? @davidtwco (rustbot has picked a reviewer for you, use r? to override) |
Thank you! r? lqd @bors r+ rollup |
The job Click to see the possible cause of the failure (guessed by this bot)
|
The test failure is unrelated to this PR: #117060. |
Let's see if closing + reopening will trigger the PR builder with the new master commit, and hopefully not have this flaky test fail. |
not that it matters, bors doesn't consider the greenness of the PR when deciding to merge. the commit is still approved: https://bors.rust-lang.org/queue/rust |
Even if it's not marked mergeable? (or rather, neither yes nor no). Maybe that would impact the automated PR rollup builder. No big deal anyways. |
i think it doesnt care, but im not sure |
I guess we'll find out soon enough since CI failed again |
The job Click to see the possible cause of the failure (guessed by this bot)
|
…iaskrgr Rollup of 5 pull requests Successful merges: - rust-lang#116244 (Apply structured suggestion that allows test to work since 1.64) - rust-lang#117686 (Build pre-coroutine-transform coroutine body on error) - rust-lang#117834 (target_feature: make it more clear what that 'Option' means) - rust-lang#117893 (Suggest dereferencing the LHS for binops such as `&T == T`) - rust-lang#117911 (Fix some typos) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#117911 - catandcoder:master, r=lqd Fix some typos
No description provided.