Rename erblint executable to erb_lint since deprecated by Shopify (#5111)
CI / Build (push) Canceled after 0s
CI / Neovim 0.10 Windows (push) Canceled after 0s
CI / Neovim 0.12 Windows (push) Canceled after 0s
CI / Vim 8.2 Windows (push) Canceled after 0s
CI / Vim 9.2 Windows (push) Canceled after 0s
CI / Lint (push) Canceled after 0s
CI / Lua (push) Canceled after 0s
CI / Neovim 0.10 Linux (push) Canceled after 0s
CI / Neovim 0.12 Linux (push) Canceled after 0s
CI / Vim 8.2 Linux (push) Canceled after 0s
CI / Vim 9.2 Linux (push) Canceled after 0s

* Rename erblint executable to erb_lint

Fix warning: "Calling `erblint` is deprecated, please call the renamed executable `erb_lint` instead."

* Update default erblint executable name

Change default value of erblint executable from 'erblint' to 'erb_lint'.

* Update tests to new excecutable default

* fix test
This commit is contained in:
Niels Bennke
2026-08-09 19:26:28 +09:00
committed by GitHub
parent 9e2efaa4d3
commit 199a95d386
3 changed files with 8 additions and 8 deletions
+2 -2
View File
@@ -2,13 +2,13 @@
" based on the ale ruumba and robocop linters
" Description: ERB Lint, support for https://github.com/Shopify/erb-lint
call ale#Set('eruby_erblint_executable', 'erblint')
call ale#Set('eruby_erblint_executable', 'erb_lint')
call ale#Set('eruby_erblint_options', '')
function! ale_linters#eruby#erblint#GetCommand(buffer) abort
let l:executable = ale#Var(a:buffer, 'eruby_erblint_executable')
return ale#ruby#EscapeExecutable(l:executable, 'erblint')
return ale#ruby#EscapeExecutable(l:executable, 'erb_lint')
\ . ' --format json '
\ . ale#Var(a:buffer, 'eruby_erblint_options')
\ . ' --stdin %s'
+1 -1
View File
@@ -40,7 +40,7 @@ erblint *ale-eruby-erblint*
eruby_erblint_executable
g:ale_eruby_erblint_executable
Type: |String|
Default: `'erblint'`
Default: `'erb_lint'`
Override the invoked erblint binary. This is useful for running erblint
from binstubs or a bundle.
+5 -5
View File
@@ -2,14 +2,14 @@ Before:
call ale#assert#SetUpLinterTest('eruby', 'erblint')
call ale#test#SetFilename('dummy.html.erb')
let g:ale_eruby_erblint_executable = 'erblint'
let g:ale_eruby_erblint_executable = 'erb_lint'
let g:ale_eruby_erblint_options = ''
After:
call ale#assert#TearDownLinterTest()
Execute(Executable should default to erblint):
AssertLinter 'erblint', ale#Escape('erblint')
Execute(Executable should default to erb_lint):
AssertLinter 'erb_lint', ale#Escape('erb_lint')
\ . ' --format json --stdin %s'
Execute(Should be able to set a custom executable):
@@ -18,9 +18,9 @@ Execute(Should be able to set a custom executable):
AssertLinter 'bin/erblint' , ale#Escape('bin/erblint')
\ . ' --format json --stdin %s'
Execute(Setting bundle appends 'exec erblint'):
Execute(Setting bundle appends 'exec erb_lint'):
let g:ale_eruby_erblint_executable = 'path to/bundle'
AssertLinter 'path to/bundle', ale#Escape('path to/bundle')
\ . ' exec erblint'
\ . ' exec erb_lint'
\ . ' --format json --stdin %s'