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

Other completion managers #12

Open
FatBoyXPC opened this issue Feb 28, 2019 · 8 comments
Open

Other completion managers #12

FatBoyXPC opened this issue Feb 28, 2019 · 8 comments

Comments

@FatBoyXPC
Copy link
Contributor

Is there any planned support for other completion managers, such as Deoplete?

@FatBoyXPC
Copy link
Contributor Author

FWIW I'd be more than happy to try and learn how to add a deoplete bridge for this but I'd probably want some direction first.

@noahfrederick
Copy link
Owner

FWIW I'd be more than happy to try and learn how to add a deoplete bridge for this but I'd probably want some direction first.

I'm definitely open to adding support, but I'm unlikely to take this on for plug-ins I don't actively use.

If you (a) understand how to register Deoplete sources (I don't) and (b) look at the existing ncm example, it should be very straightforward.

Take a look at the two existing completion sources and where they're registered. I imagine Deoplete requires a similar incantation.

@FatBoyXPC
Copy link
Contributor Author

I don't understand how to register Deoplete sources, but I'll take a look!

@sharethewisdom
Copy link

sharethewisdom commented Jul 19, 2019

I use mucomplete which cycles native completion methods. I haven't properly used your plugin, and I haven't looked at this in detail, but I imagine I'd like to set something like

let g:mucomplete#chains.php  = ['laravel-routes', 'laravel-views', 'omni', 'file']
  let g:mucomplete#can_complete.php = {
    \ 'lavarel-routes': { t -> t =~ '\broute\([''"]' },
    \ 'lavarel-view: { t -> t =~ '@(component|extends|include)\([''"]' },
    \ 'file': { t -> t =~ '\f\f$' }}

where can_complete is a "dictionary defining the conditions that must be satisfied for a given method to be tried".

It should not be too difficult to add functions for a mucomplete user method. I'll take a look when I have time.

@FatBoyXPC
Copy link
Contributor Author

FatBoyXPC commented Jul 19, 2019

I completely forgot about this issue and I've since moved to ncm2. I likely won't be writing that Deoplete bridge now.

PS: It looks like I'm not getting insert mode completion in ncm2, but I might not have something setup right. Is ncm2 expected to work with this?

@noahfrederick
Copy link
Owner

PS: It looks like I'm not getting insert mode completion in ncm2, but I might not have something setup right. Is ncm2 expected to work with this?

Unfortunately only the original NCM is supported. NCM2 support should be easy to add, but my attention has been elsewhere. I use it too, so it's something I want.

@FatBoyXPC
Copy link
Contributor Author

FatBoyXPC commented Jul 19, 2019

I got the ncm2 stuff figured out. PR incoming soon!

@noahfrederick
Copy link
Owner

NCM2 support was added thanks to @FatBoyXPC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants