-
Notifications
You must be signed in to change notification settings - Fork 13.4k
run rustfmt on libcollections test module #33795
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
Conversation
r? @aturon (rust_highfive has picked a reviewer for you, use r? to override) |
r? @Manishearth |
cc @nrc |
/////////////////////////////////////////////////////////////////////////// | ||
// intersect | ||
|
||
/// //////////////////////////////////////////////////////////////////////// |
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.
Could you manually fixup these comment changes please (to undo what rustfmt has done)
r+ with the inline comments addressed |
@nrc done. Thanks |
Thanks! @bors: r+ |
📌 Commit f6edd51 has been approved by |
@@ -220,6 +222,7 @@ fn test_overflow() { | |||
#[allow(dead_code)] | |||
#[derive(Copy, Clone)] | |||
#[repr(usize)] | |||
#[rustfmt_skip] |
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.
Seems like this caused the build problem.
The error can be seen here.
⌛ Testing commit f6edd51 with merge dc4f6e0... |
💔 Test failed - auto-mac-64-opt |
@nrc removed #[rustfmt_skip] . Please approve. Thanks |
@bors: r+ |
📌 Commit 69db6e6 has been approved by |
⌛ Testing commit 69db6e6 with merge 8fa02a5... |
⛄ The build was interrupted to prioritize another pull request. |
run rustfmt on libcollections test module
No description provided.