Skip to content

feat: Do not complete unstable items that use an internal feature#22044

Merged
Veykril merged 1 commit intorust-lang:masterfrom
ChayimFriedman2:internal-features
Apr 15, 2026
Merged

feat: Do not complete unstable items that use an internal feature#22044
Veykril merged 1 commit intorust-lang:masterfrom
ChayimFriedman2:internal-features

Conversation

@ChayimFriedman2
Copy link
Copy Markdown
Contributor

@ChayimFriedman2 ChayimFriedman2 commented Apr 14, 2026

Unless the feature is enabled (and we're on nightly).

We can choose to not complete any unstable item if its feature is not enabled. However people may first type the thing then insert the feature. Only doing that for internal features is a good middle ground: normal people don't want their completion list polluted by std::intrinsics (even if they use nightly), and std people still get their completion if the feature is enabled (and realistically, it will be).

Closes #14077.

Unless the feature is enabled (and we're on nightly).

We can choose to not complete any unstable item if its feature is not enabled. However people may first type the thing then insert the feature. Only doing that for internal features is a good middle ground: normal people don't want their completion list polluted by `std::intrinsics` (even if they use nightly), and std people still get their completion if the feature is enabled (and realistically, it will be).
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 14, 2026
@Veykril Veykril added this pull request to the merge queue Apr 15, 2026
Merged via the queue into rust-lang:master with commit 94421fe Apr 15, 2026
18 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 15, 2026
@ChayimFriedman2 ChayimFriedman2 deleted the internal-features branch April 15, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sugestion: Never suggest {core, std}::intrinsics items if the crate doesn't have feature(core_intrinsics) set.

3 participants