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

COMP: show all re-exports of the same item in completion #6650

Merged
merged 2 commits into from Jan 13, 2021

Conversation

Undin
Copy link
Member

@Undin Undin commented Jan 11, 2021

Previously, we filter out secondary import candidates for the same item from completion suggestions, as a result, if a library (e.g. async-std) re-exports some item from another library/stdlib, completion shows only single suggestion per item.

Now, we show all suggestions that can be imported - the same items as in Auto Import quick-fix

Fixes #5415

changelog: Show all re-exports of the same item in completion list. Previously, the plugin shows only one suggestion per item that hid alternative ways to import the item.

- moved all text-based methods from `RsCompletionTestFixtureBase` to `RsCompletionTestFixture`
- provide a way to pass custom renderer for `checkContainsCompletion` methods
Previously, we filter out secondary import candidates for the same item from completion suggestions, as a result, if a library (e.g. async-std) re-exports some item from another library/stdlib, completion shows only single suggestion per item.

Now, we show all suggestions that can be imported - the same items as in `Auto Import` quick-fix
@Undin Undin added the fix Pull requests that fix some bug(s) label Jan 11, 2021
Copy link
Member

@mchernyavsky mchernyavsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

@bors
Copy link
Contributor

bors bot commented Jan 13, 2021

Build succeeded:

@bors bors bot merged commit 3216c58 into master Jan 13, 2021
@bors bors bot deleted the undin/5415-improve-completion-from-index branch January 13, 2021 20:37
@github-actions github-actions bot added this to the v139 milestone Jan 13, 2021
@Undin Undin modified the milestones: v139, v140 Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Pull requests that fix some bug(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug report: code complete doesn't display std Future
2 participants