jshaver
/
ale
forked from github-mirrors/ale
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Re-apply #3538 - remove -T argument from ruby linter (#3951) 

The `-T` option (for "taint checking") was deprecated in ruby 2.7
and removed entirely in ruby 3.0. This causes the linter to fail
entirely for users of ruby 3.0.

This was reported in #3537, and then fixed in #3538 - but it seems as
though in 9fe7b1fe6a, it was accidentally
and entirely undone.

This commit is essentially identical to #3538, aside from a path change
for the tests.
This commit is contained in:
Andrew Hayworth 2021-10-22 00:20:00 -05:00 committed by GitHub
parent 9b5a3581eb
commit 95ba7898b4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ale_linters/ruby/ruby.vim
View File

@ -6,7 +6,7 @@ call ale#Set('ruby_ruby_executable', 'ruby')
call ale#linter#Define('ruby', {
\ 'name': 'ruby',
\ 'executable': {b -> ale#Var(b, 'ruby_ruby_executable')},
\ 'command': '%e -w -c -T1 %t',
\ 'command': '%e -w -c %t',
\ 'output_stream': 'stderr',
\ 'callback': 'ale#handlers#ruby#HandleSyntaxErrors',
\})

4
test/linter/test_ruby.vader
View File

@ -5,9 +5,9 @@ After:
call ale#assert#TearDownLinterTest()
Execute(The default command should be correct):
AssertLinter 'ruby', ale#Escape('ruby') . ' -w -c -T1 %t'
AssertLinter 'ruby', ale#Escape('ruby') . ' -w -c %t'
Execute(The executable should be configurable):
let g:ale_ruby_ruby_executable = 'foobar'
AssertLinter 'foobar', ale#Escape('foobar') . ' -w -c -T1 %t'
AssertLinter 'foobar', ale#Escape('foobar') . ' -w -c %t'