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 #46225

Merged
merged 10 commits into from
Nov 24, 2017
Merged

Rollup of 5 pull requests #46225

merged 10 commits into from
Nov 24, 2017

Conversation

virgil-palanciuc and others added 10 commits November 21, 2017 22:29
Also adds #![deny(missing_debug_implementations)] so they don't get
missed again.
and fix the date for 1.22.0
The idea of ‘source’ and ‘destination’ aren’t very applicable for this
operation since both slices can both be considered sources and
destinations.
Add test for rust-lang#44953

Added the requested test - trying to see if it passes; my local build fails, but not sure why - the nightly shows this output, but in my build the compilation error changed.

Fixes rust-lang#44953.
Add missing Debug impls to std_unicode

Also adds `#![deny(missing_debug_implementations)]` so they don't get missed again.

cc rust-lang#31869
Amend RELEASES for 1.22.1

and fix the date for 1.22.0
…p-param, r=kennytm

Rename param in `[T]::swap_with_slice` from `src` to `other`.

The idea of ‘source’ and ‘destination’ aren’t very applicable for this
operation since both slices can both be considered sources and
destinations.
…ennytm

Clarify stdin behavior of `Command::output`.

Fixes rust-lang#44929.
@GuillaumeGomez
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Contributor

bors commented Nov 23, 2017

📌 Commit e6968df has been approved by GuillaumeGomez

@bors
Copy link
Contributor

bors commented Nov 23, 2017

⌛ Testing commit e6968df with merge bbd7932...

bors added a commit that referenced this pull request Nov 23, 2017
Rollup of 5 pull requests

- Successful merges: #45635, #46177, #46190, #46218, #46220
- Failed merges:
@bors
Copy link
Contributor

bors commented Nov 24, 2017

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

@bors bors merged commit e6968df into rust-lang:master Nov 24, 2017
@GuillaumeGomez GuillaumeGomez deleted the rollup branch November 24, 2017 09:06
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants