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

Allow ido or none for specific use Cases: (helm-completing-read-handlers-alist) #892

Closed
jmercouris opened this Issue Feb 17, 2017 · 5 comments

Comments

Projects
None yet
3 participants
@jmercouris
Copy link

jmercouris commented Feb 17, 2017

In helm, the following is very useful:
helm-completing-read-handlers-alist

the reason for this is that ivy breaks compatibility with some packages, consider my earlier bug report: jorgenschaefer/elpy#1084

according to the author of Elpy, Elpy supports the standard interface for completion, which breaks when used in conjunction with Ivy

@jmercouris

This comment has been minimized.

Copy link
Author

jmercouris commented Feb 17, 2017

so, as is clear from the elpy thread, I think this feature is kind of necessary...

@drym3r

This comment has been minimized.

Copy link

drym3r commented Feb 17, 2017

I quoute forcer from #emacs:

ivy.el already has a facility to disable the mode for certain commands. Sadly that list is hardcoded and not extensible. You could edit ivy.el line 1749, to add elpy-doc there.

Maybe just making this editable would solve the problem.

@abo-abo abo-abo closed this in 7100925 Feb 17, 2017

@abo-abo

This comment has been minimized.

Copy link
Owner

abo-abo commented Feb 17, 2017

Thanks, I added the defcustom. But it's a hotfix rather than a solution. The purpose of a completion engine is to work, instead of not to work, and it should be trivial to make elpy completion work with ivy.

Check out https://github.com/abo-abo/lpy. It provides great inline completion for Python, among other things.

@jmercouris

This comment has been minimized.

Copy link
Author

jmercouris commented Feb 17, 2017

@drym3r

This comment has been minimized.

Copy link

drym3r commented Feb 17, 2017

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.