-
Notifications
You must be signed in to change notification settings - Fork 10
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
Compliment-like features #10
Comments
We can definitely mirror the contextual completion in ns declarations (see ns_mappings.clj in compliment). It'll take a bit of tweaking since Fuzzy matching should be easy; we just need to switch out the following line in complete.clj: (filter #(.startsWith % prefix)) |
Well, I'd love to see this for cljs and I know many other people would love to see it as well. :-) |
I actually have a branch with flx-like fuzzy completion (using Damerau-Levenshtein and a QuickSilver-like algorithm) - the problem I encountered is that emacs has These algorithms are also inherently tunable - should we let users pick the scoring thresholds? If so it might be best to return all potential candidates to the middleware and have the fuzzy-matching logic there (so the same client settings apply to both clj and cljs). The other option is to return all potential candidates to emacs and let |
I will work on the ns-form / locals contextual completion, though :-). |
Makes sense to me.
I'd suggest keeping things simple and just returning whatever
I'm definitely not a fan of this approach. |
Btw, a related ticket clojure-emacs/cider#467 |
Any progress here? |
I guess not. :-) |
@cichli is there any chance you still have this branch? |
Likely this is the only commit in this branch - cichli@352f42c |
autogenerated with https://github.com/MalloZup/doghub: issue inactive since 450 days. Please update the issue or close it |
In order to provide a similar experience for users of both Clojure and ClojureScript it'd be nice if the completion handling code supported several compliment features:
I guess those should be straightforward to port?
The text was updated successfully, but these errors were encountered: