Optimize for cm_refresh, allow triggering in the middle of a word

roxma / nvim-completion-manager

:warning: PLEASE USE https://github.com/ncm2/ncm2 INSTEAD

MIT License

917 stars 49 forks source link

Optimize for cm_refresh, allow triggering in the middle of a word #96

Closed roxma closed 7 years ago

roxma commented 7 years ago

This PR should support #60 better. It allows triggering refresh in the middle of a word. It should be more flexible than previous implementation.

@SevereOverfl0w , Would you test this PR with https://github.com/clojure-vim/async-clj-omni, removing the refresh=True, with cm_refresh_patterns=[r'/$']?

SevereOverfl0w commented 7 years ago

This doesn't seem to enable mid-word refreshing in async-clj-omni. I've tried debugging why, but can't seem to figure out how a refresh is triggered exactly.

I tried both with & without the / in the word_pattern.

roxma commented 7 years ago

@SevereOverfl0w

I've pushed another update. It should work now.

SevereOverfl0w commented 7 years ago

That works brilliantly! :tada: