fully deprecate the legacy integral modules#146882
fully deprecate the legacy integral modules#146882cyrgani wants to merge 1 commit intorust-lang:mainfrom
Conversation
This comment has been minimized.
This comment has been minimized.
|
I'm still kind of meh on these deprecations personally, but we should probably finish what was started. With that said, I'm strongly opposed to applying Once the |
438bfa3 to
7fa9c6a
Compare
This comment has been minimized.
This comment has been minimized.
|
I've removed the |
Perhaps! But the solution is definitely not to completely hide them.
Sure, same thing applies here. I don't know anything about how rustdoc's search result ranking works, but it seems reasonable that it could take deprecation status into account. |
|
@rfcbot merge |
|
Team member @BurntSushi has proposed to merge this. The next step is review by the rest of the tagged team members: No concerns currently listed. Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up! See this document for info about what commands tagged team members can give me. |
…mdev give a better example why `std` modules named like primitives are needed A small update to the `std` `lib.rs` introduction to replace mentions of `std::i32` (never needed) with `std::char` (sometimes needed). Related to rust-lang#146882.
…mdev give a better example why `std` modules named like primitives are needed A small update to the `std` `lib.rs` introduction to replace mentions of `std::i32` (never needed) with `std::char` (sometimes needed). Related to rust-lang#146882.
…mdev give a better example why `std` modules named like primitives are needed A small update to the `std` `lib.rs` introduction to replace mentions of `std::i32` (never needed) with `std::char` (sometimes needed). Related to rust-lang#146882.
give a better example why `std` modules named like primitives are needed A small update to the `std` `lib.rs` introduction to replace mentions of `std::i32` (never needed) with `std::char` (sometimes needed). Related to rust-lang/rust#146882.
|
🔔 This is now entering its final comment period, as per the review above. 🔔 |
It seems the doc(hidden) part got removed; please also update the PR description since it will permanently become part pf the git history. |
This comment has been minimized.
This comment has been minimized.
ddd855f to
5ecd92b
Compare
|
Some changes occurred in src/tools/rustfmt cc @rust-lang/rustfmt |
This comment has been minimized.
This comment has been minimized.
5ecd92b to
6002eba
Compare
|
cc @Amanieu, @folkertdev, @sayantn |
This comment has been minimized.
This comment has been minimized.
6002eba to
e7e27f5
Compare
This comment has been minimized.
This comment has been minimized.
e7e27f5 to
bde8767
Compare
|
Some changes occurred in src/tools/clippy cc @rust-lang/clippy |
bde8767 to
1767156
Compare
This comment has been minimized.
This comment has been minimized.
1767156 to
35e1fab
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
This comment has been minimized.
This comment has been minimized.
remove usages of to-be-deprecated numeric constants Split out from rust-lang#146882.
remove usages of to-be-deprecated numeric constants Split out from rust-lang#146882.
remove usages of to-be-deprecated numeric constants Split out from rust-lang#146882.
remove usages of to-be-deprecated numeric constants Split out from rust-lang#146882.
35e1fab to
a469767
Compare
a469767 to
ffe1f14
Compare
View all comments
This PR implements the last step of #68490: marking the
std::u8etc. modules and the constants instd::f32andstd::f64(but not the undeprecatedconstssubmodule) as fully deprecated.Revival of #107587 (cc @tgross35).
Closes: #68490
r? libs-api