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

Add the cquery LSP #1632

Merged
merged 3 commits into from Jun 6, 2018

Conversation

Projects
None yet
4 participants
@zed0
Copy link
Contributor

commented Jun 6, 2018

This adds the cquery LSP (https://github.com/cquery-project/cquery)
This would close #1475 and address one of the points in #1594.

@zed0 zed0 force-pushed the zed0:master branch from e71063f to 2198de0 Jun 6, 2018

@zed0 zed0 force-pushed the zed0:master branch from 2198de0 to 20db9ab Jun 6, 2018

@w0rp

w0rp approved these changes Jun 6, 2018

Copy link
Owner

left a comment

@w0rp w0rp merged commit 67753de into w0rp:master Jun 6, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@w0rp

This comment has been minimized.

Copy link
Owner

commented Jun 6, 2018

Cheers! 🍻

@andreypopp

This comment has been minimized.

Copy link
Contributor

commented Jun 24, 2018

Any reason it’s not enabled for C?

@w0rp

This comment has been minimized.

Copy link
Owner

commented Jun 24, 2018

No particular reason, the author just didn't bother. Create a pull request for it.

call ale#Set('cpp_cquery_cache_directory', expand('~/.cache/cquery'))

function! ale_linters#cpp#cquery#GetProjectRoot(buffer) abort
let l:project_root = ale#path#FindNearestFile(a:buffer, 'compile_commands.json')

This comment has been minimized.

Copy link
@polyzen

polyzen Aug 4, 2018

Contributor

.cquery as the filename is another option here. It works if I simply replace the filename in cquery.vim, but I haven't been able to figure out how to make it use either.

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.