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

Common-lisp layer slime-company integration. #7051

Closed
wants to merge 1 commit into from
Closed

Common-lisp layer slime-company integration. #7051

wants to merge 1 commit into from

Conversation

alt-vena
Copy link
Contributor

@alt-vena alt-vena commented Sep 6, 2016

Integration of slime-company contrib by default when using the common-lisp layer. Company auto complete will now work as well as all options from the slime-company (ie fuzzy matching). Also added a variable to disable/enable (enabled by default as no auto complete is working atm).

First contribution so tell me if I have done some conventions wrong and I will fix.

commit:

Add packages/init-functions. Add company conditional.

Initialize and define toggle variable.

Add packages/init-functions. Add company conditional.

Initialize and define toggle variable.
(slime-company :toogle (configuration-layer/package-usedp 'company))))

;; Hook company to common-lisp-mode
(defun common-lisp/post-init-company ())
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need an empty post-init function, it doesn't do anything.

@bmag
Copy link
Collaborator

bmag commented Sep 7, 2016

@Chream can you take a look at #7018 and make your suggestions there? It also introduces slime-company, but doesn't use a layer variable.

@alt-vena
Copy link
Contributor Author

alt-vena commented Sep 8, 2016

@bmag Thanks for the suggestions and help! Ill have a new look when I have some time. I have auto completion working now for now. Ill close this and post in the referenced PR.

@alt-vena alt-vena closed this Sep 8, 2016
@AndreaOrru
Copy link

Not sure why this has not been merged... This is so useful.

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.

3 participants