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

Compatible with ALE #45

Open
SethGower opened this issue Oct 6, 2020 · 2 comments
Open

Compatible with ALE #45

SethGower opened this issue Oct 6, 2020 · 2 comments

Comments

@SethGower
Copy link

Currently there are a list of LSP clients that this works with, but ALE isn't supported. Would this be possible to add support for? I am not sure if it would be a change here or on ALE, and not sure I would know how to do so either way. Would this be a possible enhancement?

@jackguo380
Copy link
Owner

Hi sorry for getting to this so late.

I have looked into ALE many times in the past to try to add support for it but generally have been unsuccessful. Although ALE is a pretty popular plugin I didn't look much further into it since I wasn't sure how many people actually used it as their only LSP client.

I'll probably give it shot again, but I suspect it would require changes to ALE in order for it to work so no promises.

@SethGower
Copy link
Author

That's perfectly fine. I commented that when I first found this and was using ALE at the time. I have since converted to using CoC (and only CoC) instead of ALE + Deoplete, and really like it.

A thing I have noticed, and not sure if it is a problem with this plugin, so I don't want to create an issue about it, but the updated colors don't take affect until after I save the file. Which isn't a huge problem. They also break if I run clang-format on the file, so I need to save again to update colors.

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

No branches or pull requests

2 participants