Skip to content

Conversation

@cyrgani
Copy link
Contributor

@cyrgani cyrgani commented Jan 8, 2026

Features like derive_from and derive_coerce_pointee refer to actual unstable derive macros, but the derive_eq and derive_clone_copy features are internal hacks. Rename them accordingly by adding the suffix _internals.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jan 8, 2026
@rustbot
Copy link
Collaborator

rustbot commented Jan 8, 2026

r? @jhpratt

rustbot has assigned @jhpratt.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@cyrgani
Copy link
Contributor Author

cyrgani commented Jan 8, 2026

Another question: Do we really need these pages for library features in the unstable book? Out of the 29 that are currently present, 23 just say This feature is internal to the Rust compiler and is not intended for general use.

@jhpratt
Copy link
Member

jhpratt commented Jan 9, 2026

The _internals suffix is used for a number of things in stdlib, so this seems reasonable. Personally, I don't see the need for these to be listed in the unstable book, but that's not my purview.

@bors r+ rollup

@rust-bors rust-bors bot added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 9, 2026
@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 9, 2026

📌 Commit d411919 has been approved by jhpratt

It is now in the queue for this repository.

@rust-bors rust-bors bot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 9, 2026
rust-bors bot added a commit that referenced this pull request Jan 9, 2026
Rollup of 11 pull requests

Successful merges:

 - #150272 (docs(core): update `find()` and `rfind()` examples)
 - #150385 (fix `Expr::can_have_side_effects` for `[x; N]` style array literal and binary expressions)
 - #150561 (Finish transition from `semitransparent` to `semiopaque` for `rustc_macro_transparency`)
 - #150574 (Clarify `MoveData::init_loc_map`.)
 - #150762 (Use functions more in rustdoc GUI tests)
 - #150808 (rename the `derive_{eq, clone_copy}` features to `*_internals`)
 - #150816 (Fix trait method anchor disappearing before user can click on it)
 - #150821 (tests/ui/borrowck/issue-92157.rs: Remove (bug not fixed))
 - #150829 (make attrs actually use `Target::GenericParam`)
 - #150834 (Add tracking issue for `feature(multiple_supertrait_upcastable)`)
 - #150864 (The aarch64-unknown-none target requires NEON, so the docs were wrong.)

r? @ghost
@rust-bors rust-bors bot merged commit e99f5f6 into rust-lang:main Jan 9, 2026
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Jan 9, 2026
rust-timer added a commit that referenced this pull request Jan 9, 2026
Rollup merge of #150808 - derive-internals, r=jhpratt

rename the `derive_{eq, clone_copy}` features to `*_internals`

Features like `derive_from` and `derive_coerce_pointee` refer to actual unstable derive macros, but the `derive_eq` and `derive_clone_copy` features are internal hacks. Rename them accordingly by adding the suffix `_internals`.
@cyrgani cyrgani deleted the derive-internals branch January 9, 2026 15:37
github-actions bot pushed a commit to rust-lang/rust-analyzer that referenced this pull request Jan 12, 2026
Rollup of 11 pull requests

Successful merges:

 - rust-lang/rust#150272 (docs(core): update `find()` and `rfind()` examples)
 - rust-lang/rust#150385 (fix `Expr::can_have_side_effects` for `[x; N]` style array literal and binary expressions)
 - rust-lang/rust#150561 (Finish transition from `semitransparent` to `semiopaque` for `rustc_macro_transparency`)
 - rust-lang/rust#150574 (Clarify `MoveData::init_loc_map`.)
 - rust-lang/rust#150762 (Use functions more in rustdoc GUI tests)
 - rust-lang/rust#150808 (rename the `derive_{eq, clone_copy}` features to `*_internals`)
 - rust-lang/rust#150816 (Fix trait method anchor disappearing before user can click on it)
 - rust-lang/rust#150821 (tests/ui/borrowck/issue-92157.rs: Remove (bug not fixed))
 - rust-lang/rust#150829 (make attrs actually use `Target::GenericParam`)
 - rust-lang/rust#150834 (Add tracking issue for `feature(multiple_supertrait_upcastable)`)
 - rust-lang/rust#150864 (The aarch64-unknown-none target requires NEON, so the docs were wrong.)

r? @ghost
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants