#1587 - Add deprecation warnings for old linter options
This commit is contained in:
parent
c0a279f967
commit
191967cfee
@ -3,11 +3,19 @@
|
|||||||
|
|
||||||
" CLI options
|
" CLI options
|
||||||
let g:ale_html_tidy_executable = get(g:, 'ale_html_tidy_executable', 'tidy')
|
let g:ale_html_tidy_executable = get(g:, 'ale_html_tidy_executable', 'tidy')
|
||||||
|
" remove in 2.0
|
||||||
" Look for the old _args variable first.
|
" Look for the old _args variable first.
|
||||||
|
let s:deprecation_warning_echoed = 0
|
||||||
let s:default_options = get(g:, 'ale_html_tidy_args', '-q -e -language en')
|
let s:default_options = get(g:, 'ale_html_tidy_args', '-q -e -language en')
|
||||||
let g:ale_html_tidy_options = get(g:, 'ale_html_tidy_options', s:default_options)
|
let g:ale_html_tidy_options = get(g:, 'ale_html_tidy_options', s:default_options)
|
||||||
|
|
||||||
function! ale_linters#html#tidy#GetCommand(buffer) abort
|
function! ale_linters#html#tidy#GetCommand(buffer) abort
|
||||||
|
" remove in 2.0
|
||||||
|
if exists('g:ale_html_tidy_args') && !s:deprecation_warning_echoed
|
||||||
|
execute 'echom ''Rename your g:ale_html_tidy_args setting to g:ale_html_tidy_options instead. Support for this will removed in ALE 2.0.'''
|
||||||
|
let s:deprecation_warning_echoed = 1
|
||||||
|
endif
|
||||||
|
|
||||||
" Specify file encoding in options
|
" Specify file encoding in options
|
||||||
" (Idea taken from https://github.com/scrooloose/syntastic/blob/master/syntax_checkers/html/tidy.vim)
|
" (Idea taken from https://github.com/scrooloose/syntastic/blob/master/syntax_checkers/html/tidy.vim)
|
||||||
let l:file_encoding = get({
|
let l:file_encoding = get({
|
||||||
|
@ -1,7 +1,9 @@
|
|||||||
" Author: w0rp <devw0rp@gmail.com>
|
" Author: w0rp <devw0rp@gmail.com>
|
||||||
" Description: flake8 for python files
|
" Description: flake8 for python files
|
||||||
|
|
||||||
|
" remove in 2.0
|
||||||
" Support an old setting as a fallback.
|
" Support an old setting as a fallback.
|
||||||
|
let s:deprecation_warning_echoed = 0
|
||||||
let s:default_options = get(g:, 'ale_python_flake8_args', '')
|
let s:default_options = get(g:, 'ale_python_flake8_args', '')
|
||||||
|
|
||||||
call ale#Set('python_flake8_executable', 'flake8')
|
call ale#Set('python_flake8_executable', 'flake8')
|
||||||
@ -38,6 +40,12 @@ function! ale_linters#python#flake8#VersionCheck(buffer) abort
|
|||||||
endfunction
|
endfunction
|
||||||
|
|
||||||
function! ale_linters#python#flake8#GetCommand(buffer, version_output) abort
|
function! ale_linters#python#flake8#GetCommand(buffer, version_output) abort
|
||||||
|
" remove in 2.0
|
||||||
|
if exists('g:ale_python_flake8_args') && !s:deprecation_warning_echoed
|
||||||
|
execute 'echom ''Rename your g:ale_python_flake8_args setting to g:ale_python_flake8_options instead. Support for this will removed in ALE 2.0.'''
|
||||||
|
let s:deprecation_warning_echoed = 1
|
||||||
|
endif
|
||||||
|
|
||||||
let l:cd_string = ale#Var(a:buffer, 'python_flake8_change_directory')
|
let l:cd_string = ale#Var(a:buffer, 'python_flake8_change_directory')
|
||||||
\ ? ale#path#BufferCdString(a:buffer)
|
\ ? ale#path#BufferCdString(a:buffer)
|
||||||
\ : ''
|
\ : ''
|
||||||
|
Loading…
x
Reference in New Issue
Block a user