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 11 pull requests #34508

Closed
wants to merge 24 commits into from
Closed

Conversation

royalstream and others added 24 commits June 4, 2016 14:03
Indicate how it gets created and add an example.
… r=nikomatsakis

run rustfmt on librustc_passes folder
… r=steveklabnik

Syntax coloring and more compact diagram

Two cosmetic improvements:
- New content was added a few days ago to the **Closures** chapter but it was missing rust's syntax coloring.
- Also, in the **Crates and Modules** chapter, a diagram was improved to be more symmetric and to take less space.
Traits where syntax's extra usage example more clearly

r? @steveklabnik
Add example for `std::thread::sleep`.

None
…teveklabnik

[doc] Fix links in Ownership section of the book

- Add a missing link definition for `[i32]`.
- Like `[stack]` link is pointing to `...#the-stack`, append `#the-heap` to `[heap]` link.
…Gomez

Add example with leading zeros

I was searching for this format very long. So I added an example to the prominent section.

I was thinking of putting the keyword leading in the corresponding section as well, what do you think?

r? @steveklabnik
…eGomez

Expand `std::path::Component` documentation.

Indicate how it gets created and add an example.
…st, r=GuillaumeGomez

rustdoc: Fix inlined renamed reexports in import lists

Fixes rust-lang#34473
@steveklabnik
Copy link
Member Author

@bors: r+ p=1 force

@bors
Copy link
Contributor

bors commented Jun 27, 2016

📌 Commit 991ddc5 has been approved by steveklabnik

@rust-highfive
Copy link
Collaborator

r? @Aatch

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

@bors
Copy link
Contributor

bors commented Jun 27, 2016

⌛ Testing commit 991ddc5 with merge 05f0cd2...

@jseyfried
Copy link
Contributor

@bors r- force

@bors
Copy link
Contributor

bors commented Jun 27, 2016

⛄ The build was interrupted to prioritize another pull request.

@jseyfried
Copy link
Contributor

(sorry, this conflicts with my breaking batch, #34424, and I'd really like to avoid rebasing all the merges)

@bors
Copy link
Contributor

bors commented Jun 28, 2016

☔ The latest upstream changes (presumably #34424) made this pull request unmergeable. Please resolve the merge conflicts.

@jseyfried
Copy link
Contributor

@steveklabnik Everything but #33951 is still mergeable.

@Centril Centril added the rollup A PR which is a rollup label Oct 24, 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.