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

Add docs for using deoplete with gopls completion #2231

Merged
merged 2 commits into from
Apr 15, 2019
Merged

Add docs for using deoplete with gopls completion #2231

merged 2 commits into from
Apr 15, 2019

Conversation

velovix
Copy link
Contributor

@velovix velovix commented Apr 15, 2019

This recommends configuring the "omni_patterns" parameter, which instructs deoplete to call the omnifunc when a certain regular expression is matched.

I found this solution here: Shougo/deoplete.nvim#965
The idea of adding docs for this was discussed here: #2185

This recommends configuring the "omni_patterns" parameter, which
instructs deoplete to call the omnifunc when a certain regular
expression is matched.

Found here: Shougo/deoplete.nvim#965
@bhcleek
Copy link
Collaborator

bhcleek commented Apr 15, 2019

Vim-go supports both gocode and gopls are currently supported for completion. Though gopls is used by default in Vim 8 and Neovim, it can be switched to use gocode by setting omnifunc to go#complete#GocodeComplete in an after/ftplugin/go.vim file. Is there any reason to recommend https://github.com/zchee/deoplete-go?

vim-go is now able to work with gocode natively and exposes this by
setting the omnifunc. As long as we tell deoplete to call omnifunc, no
additional layer is necessary.
@velovix
Copy link
Contributor Author

velovix commented Apr 15, 2019

Good point! I've removed mention of deoplete-go and rephrased the section to not specifically mention either completion back end.

@bhcleek bhcleek merged commit 9a4f21e into fatih:master Apr 15, 2019
@velovix velovix deleted the deoplete-gopls-docs branch January 27, 2020 23:43
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.

2 participants