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

return early from CompleteDone handler #3274

Merged
merged 1 commit into from
Aug 9, 2021

Conversation

bhcleek
Copy link
Collaborator

@bhcleek bhcleek commented Aug 9, 2021

Return early from the CompleteDone handler when omnifunc is not set to
go#complete#Complete.

This should help interactions with other plugins like coc.nvim that some
users leverage to do completion.

Relates to #3272

Return early from the CompleteDone handler when omnifunc is not set to
go#complete#Complete.

This should help interactions with other plugins like coc.nvim that some
users leverage to do completion.

Relates to fatih#3272
@daturkel
Copy link

daturkel commented Aug 9, 2021

I tested this out and confirmed it works as desired with coc in eliminating the issue in #3272 ! Thank you for the quick investigation and fix!

@bhcleek bhcleek merged commit 819cd8f into fatih:master Aug 9, 2021
@bhcleek bhcleek added this to the vim-go 1.26 milestone Aug 9, 2021
@bhcleek bhcleek deleted the completion/early-return branch August 9, 2021 15:49
bhcleek added a commit that referenced this pull request Aug 9, 2021
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