vim/ale
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #1927 from jparise/thrift-includes-default

Browse Source 
thrift: default thrift_thrift_includes to ['.']
This commit is contained in:
w0rp 2018-09-17 11:21:30 +01:00 committed by GitHub
commit a4a4bba884
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ale_linters/thrift/thrift.vim
View File

@ -2,7 +2,7 @@
call ale#Set('thrift_thrift_executable', 'thrift') call ale#Set('thrift_thrift_executable', 'thrift')
call ale#Set('thrift_thrift_generators', ['cpp']) call ale#Set('thrift_thrift_generators', ['cpp'])
call ale#Set('thrift_thrift_includes', []) call ale#Set('thrift_thrift_includes', ['.'])
call ale#Set('thrift_thrift_options', '-strict') call ale#Set('thrift_thrift_options', '-strict')
function! ale_linters#thrift#thrift#GetCommand(buffer) abort function! ale_linters#thrift#thrift#GetCommand(buffer) abort

2
doc/ale-thrift.txt
View File

@ -28,7 +28,7 @@ g:ale_thrift_thrift_generators *g:ale_thrift_thrift_generators*
g:ale_thrift_thrift_includes *g:ale_thrift_thrift_includes* g:ale_thrift_thrift_includes *g:ale_thrift_thrift_includes*
*b:ale_thrift_thrift_includes* *b:ale_thrift_thrift_includes*
Type: |List| of |String|s Type: |List| of |String|s
Default: `[]` Default: `['.']`
This list contains paths that will be searched for thrift `include` This list contains paths that will be searched for thrift `include`
directives. directives.

10
test/command_callback/test_thrift_command_callback.vader
View File

@ -23,22 +23,22 @@ After:
call ale#assert#TearDownLinterTest() call ale#assert#TearDownLinterTest()
Execute(The default command should be correct): Execute(The default command should be correct):
AssertLinter 'thrift', ale#Escape('thrift') . ' --gen cpp -strict' . b:suffix AssertLinter 'thrift', ale#Escape('thrift') . ' --gen cpp -I . -strict' . b:suffix
Execute(The executable should be configurable): Execute(The executable should be configurable):
let b:ale_thrift_thrift_executable = 'foobar' let b:ale_thrift_thrift_executable = 'foobar'
AssertLinter 'foobar', ale#Escape('foobar') . ' --gen cpp -strict' . b:suffix AssertLinter 'foobar', ale#Escape('foobar') . ' --gen cpp -I . -strict' . b:suffix
Execute(The list of generators should be configurable): Execute(The list of generators should be configurable):
let b:ale_thrift_thrift_generators = ['java', 'py:dynamic'] let b:ale_thrift_thrift_generators = ['java', 'py:dynamic']
AssertLinter 'thrift', ale#Escape('thrift') AssertLinter 'thrift', ale#Escape('thrift')
\ . ' --gen java --gen py:dynamic -strict' . b:suffix \ . ' --gen java --gen py:dynamic -I . -strict' . b:suffix
let b:ale_thrift_thrift_generators = [] let b:ale_thrift_thrift_generators = []
AssertLinter 'thrift', ale#Escape('thrift') . ' --gen cpp -strict' . b:suffix AssertLinter 'thrift', ale#Escape('thrift') . ' --gen cpp -I . -strict' . b:suffix
Execute(The list of include paths should be configurable): Execute(The list of include paths should be configurable):
let b:ale_thrift_thrift_includes = ['included/path'] let b:ale_thrift_thrift_includes = ['included/path']
@ -50,4 +50,4 @@ Execute(The string of compiler options should be configurable):
let b:ale_thrift_thrift_options = '-strict --allow-64bit-consts' let b:ale_thrift_thrift_options = '-strict --allow-64bit-consts'
AssertLinter 'thrift', ale#Escape('thrift') AssertLinter 'thrift', ale#Escape('thrift')
\ . ' --gen cpp -strict --allow-64bit-consts' . b:suffix \ . ' --gen cpp -I . -strict --allow-64bit-consts' . b:suffix