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

Replace any compe support with nvim-cmp #140

Closed
Julian opened this issue Oct 4, 2021 · 5 comments · Fixed by #263
Closed

Replace any compe support with nvim-cmp #140

Julian opened this issue Oct 4, 2021 · 5 comments · Fixed by #263
Labels
enhancement New feature or request

Comments

@Julian
Copy link
Owner

Julian commented Oct 4, 2021

nvim-compe was deprecated a bit back and replaced by nvim-cmp.

(There's also tabnine support for cmp here which may be worth trying given some success in vscode-land for Lean)

@Julian Julian added the enhancement New feature or request label Oct 4, 2021
Julian added a commit that referenced this issue Dec 31, 2021
Even though support for e.g. nvim-cmp abbreviations isn't
present yet -- see #140 -- for regular LSP completion,
use cmp.
@Karthik-Dulam
Copy link

Karthik-Dulam commented Jun 27, 2022

Currently the readme.md file conflicts on the recommended plugin to use.
Under "Installation"
Plug 'hrsh7th/nvim-cmp' " For LSP completion
And under "Features" and in the "Recommended configuration"

Abbreviation (unicode character) insertion, can also provide a nvim-compe source.

  abbreviations = {
    -- Set one of the following to true to enable abbreviations
    builtin = false, -- built-in expander
    compe = false, -- nvim-compe source

At the moment what is the recommended plugin to use of the two?

@gebner
Copy link
Collaborator

gebner commented Jun 27, 2022

I'd use builtin = true, I wonder if we should make that the default.

@Julian
Copy link
Owner Author

Julian commented Jun 27, 2022

Yeah, we should. Use the builtin expander.

@Julian
Copy link
Owner Author

Julian commented Jun 27, 2022

(I'll probably remove nvim-compe support, we can decide to add an nvim-cmp source if it turns out to be useful but I have other more important things in mind before ever thinking about it.)

@Julian
Copy link
Owner Author

Julian commented Jun 29, 2022

#263 should make this less confusing.

@Julian Julian closed this as completed in f536e74 Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants