-
Notifications
You must be signed in to change notification settings - Fork 177
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
Autocomplete ::keywords and ::alias/keywords #224
Comments
The libraries. //cc @cichli @alexander-yakushev |
Hey. This is the file to edit for CLJ: https://github.com/alexander-yakushev/compliment/blob/master/src/compliment/sources/keywords.clj |
@alexander-yakushev Now, that you're back maybe you can take a look in this as well? :-) P.S. Yeah, yeah - I know I'm an evil exploiter. :-) |
You won't trick me twice :D. I have plenty of other work to do. Maybe next time. |
Damn you, @bbatsov! shakes fist at sky @danskarda, now implemented, you should be able to use it in the next version of CIDER. |
Evil me! Thanks, @alexander-yakushev! I'll bump the dep, so that @danskarda can play with the new functionality. |
Cider autocomples both keywords and namespaced keywords. For example type
foo.bar/ba
and it will autocomplete tofoo.bar/baz
.This feature is works great and even better, it works in both CLJ and CLJS (yeah!). It helps a lot to reduce number of typos during development. Thank you!
What I would like to see is small improvement - to extend autocomplete to work with double colon keywords:
foo.bar
, I write::ba
and it will autocomplete to::baz
(because there exists a keyword:foo.bar/baz
)(require '[foo.bar :as f])
. I write::f/b
and it will autocomplete to::f/baz
, because there exists a keyword:foo.bar/baz
.Where this feature should be implemented? In cider-nrepl or in libraries for autocompletion?
The text was updated successfully, but these errors were encountered: