-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Some 1.2 relnotes #26613
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
Some 1.2 relnotes #26613
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
r? @aturon |
@@ -760,10 +813,13 @@ nsf <[email protected]> | |||
nwin <[email protected]> | |||
Oak <[email protected]> | |||
OGINO Masanori <[email protected]> | |||
OlegTsyba <[email protected]> | |||
Oliver Schneider <[email protected]> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Candidate for a mailmap entry?
Nice! r=me with a few nits |
for writing data to formatted strings, similar to [`io::Write`]. | ||
* [`fmt::Formatter`] adds 'debug builder' methods, [`debug_struct`], | ||
[`debug_tuple`], [`debug_list`], [`debug_set`], [`debug_map`]. These | ||
are used by code generators to emit implementations of [`Debug`]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe this is the right place to mention {:#?}
(pretty printing by using alternative format for Debug). It's hugely useful so we need to document it. It's a debug printer feature, only problem it was available already from Rust 1.1 in derived Debug impls.
I tagged PR #26058, so its entry could read
|
☔ The latest upstream changes (presumably #26579) made this pull request unmergeable. Please resolve the merge conflicts. |
@@ -13,7 +13,7 @@ Alex Rønne Petersen <[email protected]> | |||
Andreas Gal <[email protected]> <[email protected]> | |||
Andrew Poelstra <[email protected]> <[email protected]> | |||
Anton Löfgren <[email protected]> <[email protected]> | |||
Ariel Ben-Yehuda <[email protected]> <[email protected]> | |||
Ariel Ben-Yehuda <[email protected]> arielb1 <[email protected]> <[email protected]> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no need to have my mail twice.
6337577
to
9cbef6c
Compare
@bors r=alexcrichton |
📌 Commit 0d60e8d has been approved by |
⌛ Testing commit 0d60e8d with merge fc03764... |
💔 Test failed - auto-linux-32-opt |
@bors: retry On Thu, Jul 9, 2015 at 5:44 PM, bors [email protected] wrote:
|
⌛ Testing commit 0d60e8d with merge a5e0348... |
@bors: retry force clean |
⌛ Testing commit 0d60e8d with merge 9cb1c29... |
💔 Test failed - auto-mac-32-opt |
@bors: retry On Thu, Jul 9, 2015 at 11:50 PM, bors [email protected] wrote:
|
⌛ Testing commit 0d60e8d with merge 4695fbd... |
Haven't finished these, but since [others](#26579) are also working on this and I'm on vacation I figured I should get this in.
Haven't finished these, but since others are also working on this and I'm on vacation I figured I should get this in.