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.

Sign up for GitHub

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

Jump to bottom

Rollup of 4 pull requests #33272

Merged
merged 7 commits into from
Apr 29, 2016
Merged

Rollup of 4 pull requests #33272

merged 7 commits into from
Apr 29, 2016

Conversation

Manishearth
Copy link
Member

@rust-highfive
Copy link
Collaborator

r? @Manishearth

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

@Manishearth
Copy link
Member Author

@bors r+ p=10

@bors
Copy link
Contributor

bors commented Apr 29, 2016

📌 Commit ef3af48 has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Apr 29, 2016

⌛ Testing commit ef3af48 with merge c3e94b4...

@bors
Copy link
Contributor

bors commented Apr 29, 2016

💔 Test failed - auto-linux-64-opt-mir

@Manishearth
Rollup merge of rust-lang#33248 - jseyfried:fix_save_analysis_bug, r=nrc
13c1956 
rustc_save_analysis: fix a bug in which glob imports are not dumped

This fixes rust-lang#33213, a bug that prevents glob imports from being included in the save-analysis data.
r? @nrc
@Manishearth
Rollup merge of rust-lang#33253 - ergenekonyigit:master, r=alexcrichton
3a0efff 
rustdoc: fixed some RFCs

r? @alexcrichton
@Manishearth
Rollup merge of rust-lang#33258 - birkenfeld:fmt-dollar-syntax, r=brson
f30058f 
Clarify std::fmt width docs w.r.t. dollar syntax and give example.

The previous version only said "the `2$` syntax", which while introduced in the grammar is not very self-explanatory.
@Manishearth
Copy link
Member Author

@bors r+ force

@bors
Copy link
Contributor

bors commented Apr 29, 2016

📌 Commit f30058f has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Apr 29, 2016

⌛ Testing commit f30058f with merge e07e34c...

bors added a commit that referenced this pull request Apr 29, 2016
@bors
Auto merge of #33272 - Manishearth:rollup, r=Manishearth
e07e34c 
Rollup of 4 pull requests

- Successful merges: #33239, #33248, #33253, #33258
- Failed merges:
@bors
Copy link
Contributor

bors commented Apr 29, 2016

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-32cross-opt, auto-linux-64-cargotest, auto-linux-64-cross-armhf, auto-linux-64-cross-armsf, auto-linux-64-cross-freebsd, auto-linux-64-cross-netbsd, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-opt-mir, auto-linux-64-opt-rustbuild, auto-linux-64-x-android-t, auto-linux-cross-opt, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-mac-64-opt-rustbuild, auto-mac-cross-ios-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-32-opt-rustbuild, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-cross-opt, auto-win-msvc-32-opt, auto-win-msvc-64-cargotest, auto-win-msvc-64-opt, auto-win-msvc-64-opt-mir, auto-win-msvc-64-opt-rustbuild

@bors bors merged commit f30058f into rust-lang:master Apr 29, 2016
@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
Reviewers
No reviews
Assignees

@Manishearth Manishearth

Labels
rollup A PR which is a rollup
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Manishearth @rust-highfive @bors @Centril @ergenekonyigit @birkenfeld @jseyfried