mirror of https://github.com/dense-analysis/ale
#2397 - didChangeConfiguration should be a notification
This commit is contained in:
parent
5be6a2b4e0
commit
435a072181
|
@ -159,7 +159,7 @@ function! ale#lsp#message#Hover(buffer, line, column) abort
|
|||
endfunction
|
||||
|
||||
function! ale#lsp#message#DidChangeConfiguration(buffer, config) abort
|
||||
return [0, 'workspace/didChangeConfiguration', {
|
||||
return [1, 'workspace/didChangeConfiguration', {
|
||||
\ 'settings': a:config,
|
||||
\}]
|
||||
endfunction
|
||||
|
|
|
@ -206,7 +206,7 @@ Execute(ale#lsp#message#DidChangeConfiguration() should return correct messages)
|
|||
\ }
|
||||
AssertEqual
|
||||
\ [
|
||||
\ 0,
|
||||
\ 1,
|
||||
\ 'workspace/didChangeConfiguration',
|
||||
\ {
|
||||
\ 'settings': {
|
||||
|
|
Loading…
Reference in New Issue