Skip to content

Rollup of 5 pull requests #58714

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 10 commits into from
Feb 25, 2019
Merged

Rollup of 5 pull requests #58714

merged 10 commits into from
Feb 25, 2019

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Feb 25, 2019

Successful merges:

Failed merges:

r? @ghost

nox and others added 10 commits February 13, 2019 12:23
Notably, hash iterators don't require any trait bounds to be iterated.
Notably, iterators don't require any trait bounds to be iterated.
Relax some Hash bounds on HashMap<K, V, S> and HashSet<T, S>

Notably, hash iterators don't require any trait bounds to be iterated.
…lnay

Relax some Ord bounds on BinaryHeap<T>

Notably, iterators don't require any trait bounds to be iterated.
replace deprecated rustfmt_skip with rustfmt::skip
@Centril
Copy link
Contributor Author

Centril commented Feb 25, 2019

@bors r+ p=5

@bors
Copy link
Collaborator

bors commented Feb 25, 2019

📌 Commit 6806d0c has been approved by Centril

@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 Feb 25, 2019
@bors
Copy link
Collaborator

bors commented Feb 25, 2019

⌛ Testing commit 6806d0c with merge c1911ba...

bors added a commit that referenced this pull request Feb 25, 2019
Rollup of 5 pull requests

Successful merges:

 - #58370 (Relax some Hash bounds on HashMap<K, V, S> and HashSet<T, S>)
 - #58421 (Relax some Ord bounds on BinaryHeap<T>)
 - #58686 (replace deprecated rustfmt_skip with rustfmt::skip)
 - #58697 (Use ? in some macros)
 - #58704 (Remove some unnecessary 'extern crate')

Failed merges:

r? @ghost
@bors
Copy link
Collaborator

bors commented Feb 25, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: Centril
Pushing c1911ba to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Feb 25, 2019
@bors
Copy link
Collaborator

bors commented Feb 25, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: Centril
Pushing c1911ba to master...

@bors bors merged commit 6806d0c into rust-lang:master Feb 25, 2019
@Centril Centril deleted the rollup branch February 25, 2019 06:31
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.

5 participants