Skip to content

Fix 'No matching autocommands' warning caused by #437 #440

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

Merged
merged 2 commits into from
Dec 17, 2020
Merged

Fix 'No matching autocommands' warning caused by #437 #440

merged 2 commits into from
Dec 17, 2020

Conversation

n-p-e
Copy link
Contributor

@n-p-e n-p-e commented Oct 29, 2020

The warning only appears in Vim. I only tested on Neovim and did not notice the warning. It is fixed now.

I'm sorry for the mess.

@skamsie
Copy link

skamsie commented Nov 16, 2020

Thank you for fixing this. I have been using your branch for a week now and it's working great 👍. I would enable this by default to be honest, if user has Coc, without the need to explicitly add these two lines in your vimrc

autocmd User EasyMotionPromptBegin silent! CocDisable
autocmd User EasyMotionPromptEnd silent! CocEnable

@haya14busa
Copy link
Member

Sry for the late response.
I reverted #437, so can you add the help doc update in this p-r as well?

@n-p-e
Copy link
Contributor Author

n-p-e commented Dec 17, 2020

Sry for the late response.
I reverted #437, so can you add the help doc update in this p-r as well?

Done.

@haya14busa haya14busa merged commit d75d959 into easymotion:master Dec 17, 2020
@haya14busa
Copy link
Member

Thanks! 👍

@laggardkernel
Copy link

Seems these user events EasyMotionPrompt*implemented by doautocmd trigger folding, if there's foldlevel=0 set in the modeline. I'd like to stick to chinnkarahoi's solution

I find another guy got annoyed by the modeline parsing thing too. Pitfall of Vim: :doautocmd triggers modeline processing

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.

4 participants