Uh oh!
There was an error while loading. Please reload this page.
Conversation
rust-highfive
commented
Jun 23, 2019
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @dtolnay (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
rust-highfive
commented
Jun 23, 2019
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
ghost
commented
Jun 23, 2019
both checks should succeed once #62042 is merged, right? |
Uh oh!
There was an error while loading. Please reload this page.
rust-highfive
commented
Jun 23, 2019
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Centril
commented
Jun 23, 2019
r? @Centril |
For anyone who wishes to continue to use the Documentation: https://docs.rs/try/1.0.0/try/ When you use the crate, you get no deprecation warning. Win! |
ghost
commented
Jun 25, 2019
RELNOTES: The |
Yes, deprecated items in libstd should use Considering how widely-used this used to be, maybe deprecating it a version or two in the future is a good idea? You can just set the version number to 1.39.0 or so. |
The |
This comment has been minimized.
This comment has been minimized.
RalfJung
commented
Jun 29, 2019
Sorry, I meant the entire standard library. That includes libcore (and liballoc). |
bors
commented
Jul 7, 2019
@BO41: 🔑 Insufficient privileges: not in try users |
Uh oh!
There was an error while loading. Please reload this page.
bors
commented
Jul 7, 2019
@BO41: 🔑 Insufficient privileges: not in try users |
Centril
commented
Jul 7, 2019
You can use |
ghost
commented
Jul 8, 2019
as expected, there are more places where |
Centril
commented
Jul 8, 2019
@BO41 You can use |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
rust-highfive
commented
Jul 10, 2019
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Update src/libcore/macros.rs Co-Authored-By: Mazdak Farrokhzad <twingoow@gmail.com> Change deprecated to rustc_deprecated Update src/libcore/macros.rs Co-Authored-By: Oliver Middleton <olliemail27@gmail.com> Push deprecation version one back Allow deprecated reexport .
ghost
commented
Jul 11, 2019
Could somebody please take over this PR? I am sorry this takes so long. But I didn't think it would be so much to change. |
I'd like to give this a try, however considering I've never messed around with the rustc source before I don't think that I'm gonna be considerably faster than @BO41, however I'd still love to try! |
ghost
commented
Jul 14, 2019
Deprecate `try!` macro Replaces rust-lang#62077Fixesrust-lang/rust-clippy#1361Fixesrust-lang#61000

Depends on #62042 being merged first
Fixesrust-lang/rust-clippy#1361
Fixes#61000