Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable RsReferenceCodeVisionProvider for top-level items #10763

Merged
merged 2 commits into from Jul 30, 2023

Conversation

vlad20012
Copy link
Member

The provider was initially added in #9927.

Here I enable usages Code Vision for Rust items except:

  • an associated item (an impl/trait member)
  • a struct field
  • a macro declaration
  • an enum variant

Usage code vision can be enabled for these elements using org.rust.code.vision.usage.slow registry option.

For other items Reference Search does not use type inference and hence it should work fast.
I additionally adjusted RsReferenceImpl.isReferenceTo() to ensure it does not touch type inference in such cases.

changelog: Enable Usages Code Vision hints for some top-level items

@vlad20012 vlad20012 self-assigned this Jul 30, 2023
@intellij-rust-bot intellij-rust-bot added this to In Progress in To test Jul 30, 2023
@vlad20012
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Jul 30, 2023

Merge conflict.

vlad20012 and others added 2 commits July 30, 2023 20:13
Enable it for items except:
- an associated item (an impl/trait member)
- a struct field
- a macro declaration
- an enum variant

Usage code vision can be enabled for these elements using
 `org.rust.code.vision.usage.slow` registry option

[skip ci]
@vlad20012 vlad20012 force-pushed the enable-usages-code-vision-for-top-level-items branch from ac588c4 to 4d1fdea Compare July 30, 2023 18:14
@vlad20012
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Jul 30, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 0742361 into master Jul 30, 2023
1 check passed
To test automation moved this from In Progress to Test Jul 30, 2023
@bors bors bot deleted the enable-usages-code-vision-for-top-level-items branch July 30, 2023 20:24
@github-actions github-actions bot added this to the v200 milestone Jul 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
To test
  
Test
Development

Successfully merging this pull request may close these issues.

None yet

1 participant