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

Better LSP completions inside of backticks #22555

Merged
merged 1 commit into from
Feb 9, 2025

Conversation

harpocrates
Copy link
Contributor

This improves the presentation compiler name completions inside of backticks. The existing gaps which motivate doing this are outlined in this Metals feature request.

This improves the presentation compiler name completions inside of
backticks. The existing gaps which motivate doing this are outlined in
[this Metals feature request][0].

[0]: scalameta/metals-feature-requests#418
Copy link
Contributor

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM

@tgodzik tgodzik merged commit a6a486e into scala:main Feb 9, 2025
29 checks passed
@harpocrates harpocrates deleted the backticked-completions branch February 9, 2025 20:10
harpocrates added a commit to harpocrates/metals that referenced this pull request Feb 10, 2025
Scala 2 only - the [corresponding Scala 3 change][0] has already been
merged. Completions can now be triggered from inside of backticks.

In addition, the backtick itself is added as a completion character.
Especially when the editor is configured to auto-close backticks, this
plays nicely with completions now producing useful results inside of
backticks.

Closes [this feature request][1]

[0]: scala/scala3#22555
[1]: scalameta/metals-feature-requests#418
harpocrates added a commit to harpocrates/metals that referenced this pull request Feb 10, 2025
Scala 2 only - the [corresponding Scala 3 change][0] has already been
merged. Completions can now be triggered from inside of backticks.

In addition, the backtick itself is added as a completion character.
Especially when the editor is configured to auto-close backticks, this
plays nicely with completions now producing useful results inside of
backticks.

Closes [this feature request][1]

[0]: scala/scala3#22555
[1]: scalameta/metals-feature-requests#418
harpocrates added a commit to harpocrates/metals that referenced this pull request Feb 10, 2025
Scala 2 only - the [corresponding Scala 3 change][0] has already been
merged. Completions can now be triggered from inside of backticks.

In addition, the backtick itself is added as a completion character.
Especially when the editor is configured to auto-close backticks, this
plays nicely with completions now producing useful results inside of
backticks.

Closes [this feature request][1]

[0]: scala/scala3#22555
[1]: scalameta/metals-feature-requests#418
harpocrates added a commit to harpocrates/metals that referenced this pull request Feb 10, 2025
Scala 2 only - the [corresponding Scala 3 change][0] has already been
merged. Completions can now be triggered from inside of backticks.

In addition, the backtick itself is added as a completion character.
Especially when the editor is configured to auto-close backticks, this
plays nicely with completions now producing useful results inside of
backticks.

Closes [this feature request][1]

[0]: scala/scala3#22555
[1]: scalameta/metals-feature-requests#418
tgodzik pushed a commit to scalameta/metals that referenced this pull request Feb 11, 2025
Scala 2 only - the [corresponding Scala 3 change][0] has already been
merged. Completions can now be triggered from inside of backticks.

In addition, the backtick itself is added as a completion character.
Especially when the editor is configured to auto-close backticks, this
plays nicely with completions now producing useful results inside of
backticks.

Closes [this feature request][1]

[0]: scala/scala3#22555
[1]: scalameta/metals-feature-requests#418
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.

2 participants