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

LSP: Show fn/value signature instead of a module name #3322

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Zhomart
Copy link

@Zhomart Zhomart commented Jun 23, 2024

RFC: What do you think about this proposal?

Motivation: I want to see the function or value signatures when autocompleting, it allows quickly understand what function does and what does it return. The module name can be deduced by the prefix before ".".

Before After
image image
Before After
image image
Before After
image image

@lpil
Copy link
Member

lpil commented Jun 23, 2024

How does the programmer know which module each item comes from after this case? The type information was also already included in the completion information, typically shown for the selected option.

The Rust example you've shown in the issue priorities the module also, and modules are less important in Rust than Gleam, so we have more need to show them.

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.

None yet

2 participants