diff options
| author | Andrew Hayworth <ahayworth@gmail.com> | 2021-10-22 00:20:00 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-10-22 14:20:00 +0900 |
| commit | 95ba7898b45c3cbac77463aace447ead4e173556 (patch) | |
| tree | fcd8917c2059fb9a382d98eef1031258a7b21d8b | |
| parent | 9b5a3581ebb7e6e1ccb0cbaeef5358aee57d734e (diff) | |
| download | ale-95ba7898b45c3cbac77463aace447ead4e173556.zip | |
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 9fe7b1fe6a23fb55e6d782293585d58193123f59, it was accidentally
and entirely undone.
This commit is essentially identical to #3538, aside from a path change
for the tests.
| -rw-r--r-- | ale_linters/ruby/ruby.vim | 2 | ||||
| -rw-r--r-- | test/linter/test_ruby.vader | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/ale_linters/ruby/ruby.vim b/ale_linters/ruby/ruby.vim index 2dc55eb0..621fcbc0 100644 --- a/ale_linters/ruby/ruby.vim +++ b/ale_linters/ruby/ruby.vim @@ -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', \}) diff --git a/test/linter/test_ruby.vader b/test/linter/test_ruby.vader index b6bac14f..d957079d 100644 --- a/test/linter/test_ruby.vader +++ b/test/linter/test_ruby.vader @@ -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' |