Skip to content
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

Array trait impl comment/doc fixes #78716

Merged
merged 2 commits into from
Nov 5, 2020
Merged

Conversation

est31
Copy link
Member

@est31 est31 commented Nov 4, 2020

Two small doc/comment fixes regarding trait implementations on arrays.

@rust-highfive
Copy link
Collaborator

r? @kennytm

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 4, 2020
@Dylan-DPC-zz
Copy link

r? @Dylan-DPC

@bors r+ rollup=always

@bors
Copy link
Contributor

bors commented Nov 4, 2020

📌 Commit eb3fd6b20a33595da6a18a53b5c6c02c70d5c378 has been approved by Dylan-DPC

@rust-highfive rust-highfive assigned Dylan-DPC-zz and unassigned kennytm Nov 4, 2020
@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 4, 2020
@bors
Copy link
Contributor

bors commented Nov 4, 2020

⌛ Testing commit eb3fd6b20a33595da6a18a53b5c6c02c70d5c378 with merge 80aa3340fa189e55053149db3863122a47a54b26...

@bors
Copy link
Contributor

bors commented Nov 4, 2020

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Nov 4, 2020
@Dylan-DPC-zz
Copy link

failed spuriously

@bors retry

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 4, 2020
@bors
Copy link
Contributor

bors commented Nov 4, 2020

⌛ Testing commit eb3fd6b20a33595da6a18a53b5c6c02c70d5c378 with merge 29709bc424baef2e2f48b5c72ceb67883ad727b6...

library/core/src/array/mod.rs Outdated Show resolved Hide resolved
@bors
Copy link
Contributor

bors commented Nov 4, 2020

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Nov 4, 2020
@camelid
Copy link
Member

camelid commented Nov 4, 2020

Another spurious network failure :/

@camelid camelid added the A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools label Nov 4, 2020
@est31
Copy link
Member Author

est31 commented Nov 4, 2020

@camelid will the PR be reapproved if I change it after it has been approved? I've done that with my other PR and there is no reaction since: #78465

@est31
Copy link
Member Author

est31 commented Nov 4, 2020

@bors retry

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 4, 2020
The comment has become outdated as the array_impl macro
has been removed.
@est31
Copy link
Member Author

est31 commented Nov 4, 2020

Shrug, I'll just try it. r? @camelid

@rust-highfive rust-highfive assigned camelid and unassigned Dylan-DPC-zz Nov 4, 2020
@Dylan-DPC-zz
Copy link

@bors r+

@bors
Copy link
Contributor

bors commented Nov 4, 2020

📌 Commit 93fa023 has been approved by Dylan-DPC

@camelid
Copy link
Member

camelid commented Nov 4, 2020

@camelid will the PR be reapproved if I change it after it has been approved? I've done that with my other PR and there is no reaction since: #78465

@est31 No, bors will not merge it if it has been changed since being approved. I think it's a security feature.

(Btw, I don't have r+ privileges, so unfortunately I wouldn't have been able to approve this.)

@camelid camelid assigned Dylan-DPC-zz and unassigned camelid Nov 4, 2020
@Dylan-DPC-zz
Copy link

don't worry i have reapproved it, so it is in the queue

@camelid
Copy link
Member

camelid commented Nov 4, 2020

Yeah, I just re-assigned you so that if there are issues you can re-approve it :)

JohnTitor added a commit to JohnTitor/rust that referenced this pull request Nov 5, 2020
Array trait impl comment/doc fixes

Two small doc/comment fixes regarding trait implementations on arrays.
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Nov 5, 2020
Array trait impl comment/doc fixes

Two small doc/comment fixes regarding trait implementations on arrays.
m-ou-se added a commit to m-ou-se/rust that referenced this pull request Nov 5, 2020
Array trait impl comment/doc fixes

Two small doc/comment fixes regarding trait implementations on arrays.
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 5, 2020
Rollup of 15 pull requests

Successful merges:

 - rust-lang#76718 (Move Vec UI tests to unit tests when possible)
 - rust-lang#78093 (Clean up docs for 'as' keyword)
 - rust-lang#78425 (Move f64::NAN ui tests into `library`)
 - rust-lang#78465 (Change as_str → to_string in proc_macro::Ident::span() docs)
 - rust-lang#78584 (Add keyboard handling to the theme picker menu)
 - rust-lang#78716 (Array trait impl comment/doc fixes)
 - rust-lang#78727 ((rustdoc) fix test for trait impl display)
 - rust-lang#78733 (fix a couple of clippy warnings:)
 - rust-lang#78735 (Simplify the implementation of `get_mut` (no unsafe))
 - rust-lang#78738 (Move range in ui test to ops test in library/core)
 - rust-lang#78739 (Fix ICE on type error in async function)
 - rust-lang#78742 (make intern_const_alloc_recursive return error)
 - rust-lang#78756 (Update cargo)
 - rust-lang#78757 (Improve and clean up some intra-doc links)
 - rust-lang#78758 (Fixed typo in comment)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 43e1b58 into rust-lang:master Nov 5, 2020
@rustbot rustbot added this to the 1.49.0 milestone Nov 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants