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

Rollup of 5 pull requests #52245

Merged
merged 14 commits into from
Jul 11, 2018
Merged

Rollup of 5 pull requests #52245

merged 14 commits into from
Jul 11, 2018

Conversation

GuillaumeGomez
Copy link
Member

Successful merges:

Failed merges:

r? @ghost

@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=5

@bors
Copy link
Contributor

bors commented Jul 11, 2018

📌 Commit 985c5a7 has been approved by GuillaumeGomez

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 11, 2018
@bors
Copy link
Contributor

bors commented Jul 11, 2018

⌛ Testing commit 985c5a7 with merge 989fa05...

bors added a commit that referenced this pull request Jul 11, 2018
Rollup of 5 pull requests

Successful merges:

 - #51701 (Better docs for copy_from_slice & clone_from_slice)
 - #52231 (Fix typo in error message E0277)
 - #52233 (Improve lint handling in rustdoc)
 - #52238 (Avoid unwrapping in PanicInfo doc example.)
 - #52241 (Fix typo in E0433 docs)

Failed merges:

r? @ghost
@bors
Copy link
Contributor

bors commented Jul 11, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: GuillaumeGomez
Pushing 989fa05 to master...

@bors bors merged commit 985c5a7 into rust-lang:master Jul 11, 2018
@GuillaumeGomez GuillaumeGomez deleted the rollup branch July 11, 2018 13:39
@anirudhb
Copy link
Contributor

@GuillaumeGomez Just curious, why is it better to rollup the pull requests and then merge them, instead of merging them individually? Either way, the commits will be merged, right?

@GuillaumeGomez
Copy link
Member Author

Because like this, we merge them all at once instead of having to wait 3+ hours for each.

@anirudhb
Copy link
Contributor

Huh, bors seems to be really slow...

@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup 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