Skip to content

check_unsafety: more code reuse - #68588

Merged
bors merged 1 commit into
rust-lang:masterfrom
Centril:check-unsafety-clean
Jan 31, 2020
Merged

check_unsafety: more code reuse#68588
bors merged 1 commit into
rust-lang:masterfrom
Centril:check-unsafety-clean

Conversation

@Centril

@CentrilCentril commented Jan 28, 2020

Copy link
Copy Markdown
Contributor

r? oli-obk

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 28, 2020
@Mark-Simulacrum

Copy link
Copy Markdown
Member

Looks like this is equivalent in practice, so @bors r+ rollup

r? @Mark-Simulacrum

(also edited the parent to strip out @ to avoid pinging Oli on all merges)

@bors

bors commented Jan 30, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit dc17f38 has been approved by Mark-Simulacrum

@borsbors 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 Jan 30, 2020
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jan 31, 2020
…k-Simulacrum
check_unsafety: more code reuse
r? oli-obk
bors added a commit that referenced this pull request Jan 31, 2020
Rollup of 6 pull requests
Successful merges:
- #68588 (check_unsafety: more code reuse)
- #68638 (Add missing links for cmp traits)
- #68660 (Fix typo.)
- #68669 (suggest adding space in accidental doc comments)
- #68670 (clarify "incorrect issue" error)
- #68680 (Add `#![doc(html_playground_url = ...)]` to alloc crate)
Failed merges:
r? @ghost
@bors
bors merged commit dc17f38 into rust-lang:masterJan 31, 2020
@Centril
Centril deleted the check-unsafety-clean branch January 31, 2020 07:40
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: 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.

5 participants

@Centril@Mark-Simulacrum@bors@oli-obk@rust-highfive