summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIdan Arye <IdanArye@gmail.com>2017-03-27 14:37:00 +0300
committerw0rp <w0rp@users.noreply.github.com>2017-03-27 12:37:00 +0100
commit2f40953e037eb0d70f8052e6b035758ec3fc0d5c (patch)
tree36e9254769c972246bab63dd237550e257953d22
parent8ca82af4ab9e1d95f8b577eba68f2b443b7dba07 (diff)
downloadale-2f40953e037eb0d70f8052e6b035758ec3fc0d5c.zip
Make cargo linter lighter: (#408)
- Use `cargo check` instead of `cargo build` when `g:ale_rust_cargo_use_check` is truish. - Use `--frozen` to avoid locking the project.
-rw-r--r--ale_linters/rust/cargo.vim12
-rw-r--r--doc/ale.txt16
2 files changed, 25 insertions, 3 deletions
diff --git a/ale_linters/rust/cargo.vim b/ale_linters/rust/cargo.vim
index 7f821d2c..30600cf8 100644
--- a/ale_linters/rust/cargo.vim
+++ b/ale_linters/rust/cargo.vim
@@ -1,6 +1,8 @@
" Author: Daniel Schemala <istjanichtzufassen@gmail.com>
" Description: rustc invoked by cargo for rust files
+let g:ale_rust_cargo_use_check = get(g:, 'ale_rust_cargo_use_check', 0)
+
function! ale_linters#rust#cargo#GetCargoExecutable(bufnr) abort
if ale#util#FindNearestFile(a:bufnr, 'Cargo.toml') !=# ''
return 'cargo'
@@ -11,10 +13,18 @@ function! ale_linters#rust#cargo#GetCargoExecutable(bufnr) abort
endif
endfunction
+function! ale_linters#rust#cargo#GetCommand(buffer) abort
+ let l:command = g:ale_rust_cargo_use_check
+ \ ? 'check'
+ \ : 'build'
+
+ return 'cargo ' . l:command . ' --frozen --message-format=json -q'
+endfunction
+
call ale#linter#Define('rust', {
\ 'name': 'cargo',
\ 'executable_callback': 'ale_linters#rust#cargo#GetCargoExecutable',
-\ 'command': 'cargo build --message-format=json -q',
+\ 'command_callback': 'ale_linters#rust#cargo#GetCommand',
\ 'callback': 'ale#handlers#rust#HandleRustErrors',
\ 'output_stream': 'stdout',
\})
diff --git a/doc/ale.txt b/doc/ale.txt
index fb71de4e..efe6cd41 100644
--- a/doc/ale.txt
+++ b/doc/ale.txt
@@ -43,7 +43,8 @@ CONTENTS *ale-contents*
4.31. yamllint........................|ale-linter-options-yamllint|
4.32. cmakelint.......................|ale-linter-options-cmakelint|
4.33. perl-perl.......................|ale-linter-options-perl-perl|
- 4.34. asm-gcc.........................|ale-linter-options-asm-gcc|
+ 4.34. rust-cargo......................|ale-linter-options-rust-cargo|
+ 4.35. asm-gcc.........................|ale-linter-options-asm-gcc|
5. Linter Integration Notes.............|ale-linter-integration|
5.1. merlin..........................|ale-linter-integration-ocaml-merlin|
5.2. rust.............................|ale-integration-rust|
@@ -1123,7 +1124,18 @@ g:ale_perl_perl_options *g:ale_perl_perl_options*
invocation.
-------------------------------------------------------------------------------
-4.34. asm-gcc *ale-linter-options-asm-gcc*
+4.34. rust-cargo *ale-linter-options-rust-cargo*
+
+g:ale_rust_cargo_use_check *g:ale_rust_cargo_use_check*
+
+ Type: |Number|
+ Default: `1`
+
+ When set to `1`, this option will cause ALE to use "cargo check" instead of
+ "cargo build". "cargo check" is supported since version 1.16.0 of Rust.
+
+-------------------------------------------------------------------------------
+4.35. asm-gcc *ale-linter-options-asm-gcc*
g:ale_asm_gcc_options *g:ale_asm_gcc_options*