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: do not provide autocompletions within comments #2161

Open
Lunarmagpie opened this issue May 21, 2023 · 1 comment · May be fixed by #3156
Open

LSP: do not provide autocompletions within comments #2161

Lunarmagpie opened this issue May 21, 2023 · 1 comment · May be fixed by #3156
Labels
help wanted Contributions encouraged priority:medium

Comments

@Lunarmagpie
Copy link
Contributor

Lunarmagpie commented May 21, 2023

Heres a screenshot of this issue
image

Rust's LSP and pylance do not have this behavior. I think that is what gleam should do.

I am on gleam version nightly-230517 installed through asdf.

@Lunarmagpie Lunarmagpie changed the title LSP autocomplete has suggestions when I am writing a comment LSP autocomplete has suggestions when I am writing a comment (nightly-230517) May 21, 2023
@lpil
Copy link
Member

lpil commented May 21, 2023

Thanks for the report! The LSP does not yet understand comments, further work on understanding the position of the cursor within code is planned 🙏

@lpil lpil changed the title LSP autocomplete has suggestions when I am writing a comment (nightly-230517) LSP: do not provide autocompletions within comments May 21, 2023
@lpil lpil added help wanted Contributions encouraged area:tooling labels May 21, 2023
@lpil lpil added this to the Gleam v0.30 milestone May 22, 2023
@lpil lpil modified the milestones: Gleam v0.30, Gleam v0.31 Jun 13, 2023
@lpil lpil removed this from the Gleam v0.31 milestone Jul 15, 2023
@nicklimmm nicklimmm linked a pull request May 18, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Contributions encouraged priority:medium
Projects
Status: Unfinished
Development

Successfully merging a pull request may close this issue.

2 participants