mirror of
https://github.com/dense-analysis/ale.git
synced 2025-12-06 12:44:23 +08:00
add: support config option for checkmake linter (#4351)
`checkmake` by default checks config file "in the same folder it's executed in" unless `--config` option is set. This commit allows setting the `--config` option with an option variable (with documentation updated).
This commit is contained in:
39
test/linter/test_checkmake.vader
Normal file
39
test/linter/test_checkmake.vader
Normal file
@@ -0,0 +1,39 @@
|
||||
Before:
|
||||
Save g:ale_make_checkmake_config
|
||||
|
||||
let g:ale_make_checkmake_config = ''
|
||||
|
||||
call ale#assert#SetUpLinterTest('make', 'checkmake')
|
||||
|
||||
" NOTE: the format string must be manually matched that set in the plugin
|
||||
let b:format = '"{{.LineNumber}}:{{.Rule}}:{{.Violation}}{{\"\r\n\"}}"'
|
||||
|
||||
After:
|
||||
Restore
|
||||
|
||||
unlet! b:command_tail
|
||||
unlet! b:ale_make_checkmake_config
|
||||
|
||||
call ale#assert#TearDownLinterTest()
|
||||
|
||||
Execute(checkmake should run with default format option):
|
||||
let b:command_tail = ' --format=' . b:format . ' %s'
|
||||
|
||||
AssertLinter 'checkmake', 'checkmake' . b:command_tail
|
||||
|
||||
Execute(checkmake command should take the config option if it is non-empty):
|
||||
let g:ale_make_checkmake_config = '/path to/checkmake.ini'
|
||||
let b:command_tail = ' --format=' . b:format
|
||||
\ . ' --config="' . g:ale_make_checkmake_config . '"'
|
||||
\ . ' %s'
|
||||
|
||||
AssertLinter 'checkmake', 'checkmake' . b:command_tail
|
||||
|
||||
Execute(the local buffer config option takes precedence over global option):
|
||||
let g:ale_make_checkmake_config = '/path/to/checkmake.ini'
|
||||
let b:ale_make_checkmake_config = '/another/checkmake.ini'
|
||||
let b:command_tail = ' --format=' . b:format
|
||||
\ . ' --config="' . b:ale_make_checkmake_config . '"'
|
||||
\ . ' %s'
|
||||
|
||||
AssertLinter 'checkmake', 'checkmake' . b:command_tail
|
||||
Reference in New Issue
Block a user