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

Change completion start to 2 for nvim #24

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

Conversation

tetigi
Copy link

@tetigi tetigi commented Jan 5, 2020

The current complete_length of 0 is overwhelming and constantly bombarding the autocomplete with every possible suggestion in the namespace, even when your cursor is just chilling out as you think about what to write next. Setting it to 2 makes a bit more sense as a default option.

@SevereOverfl0w
Copy link
Member

Is there a user configuration place for this? 0 is technically what we are capable of, unless ncm2 provides specific guidance on this?

@tetigi
Copy link
Author

tetigi commented Jan 5, 2020

I had a short dig, but couldn't immediately surface an obvious way to change it (although I did not look particularly hard and I'm definitely not a ncm expert). I suppose it is a matter of opinion, but when the suggestion was popping up when I hadn't written anything, it would suggest every possibility, which isn't particularly helpful.

@tetigi
Copy link
Author

tetigi commented Jan 5, 2020

You're right that it can be set in user settings, but I still think 2 or 3 is worth considering as a sensible default, as this is what ncm is by default for high priority completers. (see https://github.com/ncm2/ncm2/blob/160e2e33d515f67154840844d8304722b679df90/doc/ncm2.txt#L221)

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.

None yet

2 participants