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

Javascript function parameters completion #3498

Closed
2 of 3 tasks
kuator opened this issue Apr 30, 2022 · 8 comments · Fixed by #3766
Closed
2 of 3 tasks

Javascript function parameters completion #3498

kuator opened this issue Apr 30, 2022 · 8 comments · Fixed by #3766
Labels

Comments

@kuator
Copy link

kuator commented Apr 30, 2022

Thank you for the bug report

  • I am using the latest version of lsp-mode related packages.
  • I checked FAQ and Troubleshooting sections
  • You may also try reproduce the issue using clean environment using the following command: M-x lsp-start-plain

Bug description

typescript-language-server added function parameters completion: typescript-language-server/typescript-language-server#303. Setting the emacs-lsp's lsp-javascript-suggest-complete-function-calls setting doesn't seem to change anything.

Steps to reproduce

  (use-package lsp-mode
    :config
    (add-hook 'c++-mode-hook #'lsp)
    (setq
     lsp-javascript-suggest-complete-function-calls t
     lsp-typescript-suggest-complete-function-calls t)
    :custom
    (lsp-enable-snippet t))

Expected behavior

Function parameters getting completed

Which Language Server did you use?

typescript-language-server

OS

Linux

Error callstack

For some reason, the log becomes 200 000 lines long after completion and I can't paste it

Anything else?

I managed to make it work with neovim

@kuator kuator added the bug label Apr 30, 2022
@yyoncho
Copy link
Member

yyoncho commented May 1, 2022

Hey @shivjm, willing to take a look?

@shivjm
Copy link
Contributor

shivjm commented May 1, 2022

Yup, happy to!

@shivjm
Copy link
Contributor

shivjm commented May 2, 2022

The settings are being correctly propagated from lsp-mode to the server, but the server never returns this kind of suggestion. The suggestion mechanism does seem to work for automatic imports, so it’s specific to this kind of completion. I’ll keep digging.

@kuator
Copy link
Author

kuator commented May 2, 2022

I'm not sure if it helps, but here's a discussion of the functionality's behaviour: hrsh7th/nvim-cmp#646 (comment)

@kuator
Copy link
Author

kuator commented May 10, 2022

Thank you!

@kuator kuator closed this as completed May 10, 2022
@yyoncho yyoncho reopened this May 10, 2022
@yyoncho
Copy link
Member

yyoncho commented May 10, 2022

I will keep it open - we might think of some hacky solution to avoid server bugs...

@davidmicah123
Copy link

yyoncho added a commit to yyoncho/lsp-mode that referenced this issue Oct 18, 2022
yyoncho added a commit that referenced this issue Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants