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.

Sign up for GitHub

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

Jump to bottom

improvement: Make including detail in completion label configurable #6986

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

improvement: Make including detail in completion label configurable #6986

wants to merge 1 commit into from
+28 −4

Commits on Nov 29, 2024

  1. improvement: Make including detail in completion label configurable 

    Some LSP clients (such as Emacs using Eglot and Corfu) use the label field to insert a completion if
completionItem/resolve has not returned by the time the completion is selected. Including the contents of the detail
property in the label causes it to be included in the file itself in these clients. To remedy this, we define a
isDetailIncludedInLabel option under compilerOptions to make this behaviour toggleable. We also disable this behaviour
by default in Emacs, since this is where the undesirable behaviour was located.

Fixes scalameta#6849.
    @Sorixelle
    Sorixelle committed Nov 29, 2024
    Configuration menu
    Copy the full SHA
    041ab36 View commit details
    Browse the repository at this point in the history