Skip to content

Conversation

@rust-highfive
Copy link
Contributor

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
Collaborator

bors commented Apr 29, 2016

📌 Commit ef3af48 has been approved by Manishearth

@bors
Copy link
Collaborator

bors commented Apr 29, 2016

⌛ Testing commit ef3af48 with merge c3e94b4...

@bors
Copy link
Collaborator

bors commented Apr 29, 2016

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

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
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
Collaborator

bors commented Apr 29, 2016

📌 Commit f30058f has been approved by Manishearth

@bors
Copy link
Collaborator

bors commented Apr 29, 2016

⌛ Testing commit f30058f with merge e07e34c...

bors added a commit that referenced this pull request Apr 29, 2016
Rollup of 4 pull requests

- Successful merges: #33239, #33248, #33253, #33258
- Failed merges:
@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
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants