Skip to content

feat: support hover on cfg predicate - #23390

Open
A4-Tacks wants to merge 3 commits into
rust-lang:masterfrom
A4-Tacks:hover-cfg-pred
Open

A4-Tacks wants to merge 3 commits into
rust-lang:masterfrom
A4-Tacks:hover-cfg-pred

Conversation

@A4-Tacks

Copy link
Copy Markdown
Member

This solves the problem of #23384 (comment)

Example

#[cfg($0test)]
fn test() {}

Before this PR

hover not found

After this PR

cfg predicate evaluated: true (test)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 18, 2026
Example
---
```rust
#[cfg($0test)]
fn test() {}
```

**Before this PR**

hover not found

**After this PR**

cfg predicate evaluated: `true` (`test`)
@rustbot

rustbot commented Sep 20, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different master 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.

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants