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

Disable code completion for default params #64

Closed
JakeBecker opened this issue Mar 12, 2018 · 3 comments
Closed

Disable code completion for default params #64

JakeBecker opened this issue Mar 12, 2018 · 3 comments

Comments

@JakeBecker
Copy link
Owner

From @quolpr on March 12, 2018 8:29

Is it possible to disable code completion for default params? For example, when I auto complete this:

Repo.delete_

It comes to this:

Repo.delete_all(queryable, opts \\ [])

But in almost all cases I don't want touch the default values

Copied from original issue: JakeBecker/vscode-elixir-ls#56

@JakeBecker
Copy link
Owner Author

Agreed, the current way of handling these is not ideal. Better, I think would be to have multiple suggestions and prioritize the ones without default params above the ones with them.

@akash-akya
Copy link

We can split the functions with default arguments into multiple functions by removing default arguments. And prioritizing functions with the least arguments first. Currently I have working crude version of this implemented in my elixir_sense fork https://github.com/akash-akya/elixir_sense.

It seems that the upstream is not actively maintained, so it might just stay in my fork.

@JakeBecker
Copy link
Owner Author

This project has moved!

It's now being maintained by proactive volunteers from the Elixir community over at elixir-lsp/elixir-ls. Updates will continue to be published from that repo to the original VS Code extension, so no need to switch plugins if you're using VS Code.

To avoid inundating the new maintainers with issues, please verify that your issue persists with the latest version of the extension (which is published from the new repo) before re-filing your issue there.

Thanks for using ElixirLS!

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

No branches or pull requests

2 participants