VimPlug/ale
1
0
Fork 0
mirror of https://github.com/dense-analysis/ale.git synced 2025-12-06 12:44:23 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix linting with jq (#4765) (#4862)
Some checks failed
CI / build_image (push) Has been cancelled
Details
CI / test_ale (--linters-only) (push) Has been cancelled
Details
CI / test_ale (--neovim-06-only) (push) Has been cancelled
Details
CI / test_ale (--neovim-08-only) (push) Has been cancelled
Details
CI / test_ale (--vim-80-only) (push) Has been cancelled
Details
CI / test_ale (--vim-90-only) (push) Has been cancelled
Details

Browse Source 
With the 1.6 version of jq the error message start with "parse error".
With the last version of jq the error message start with "jq: parse error".
Fix it by using a regular expression that works in both cases.
This commit is contained in:
benjos1234
2024-11-23 10:42:22 +01:00
committed by GitHub
parent 1e1604be56
commit 9d30fb2f59
2 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
ale_linters/json/jq.vim
View File

@@ -5,7 +5,7 @@ call ale#Set('json_jq_filters', '.')
" Matches patterns like the following: " Matches patterns like the following:
" parse error: Expected another key-value pair at line 4, column 3 " parse error: Expected another key-value pair at line 4, column 3
let s:pattern = '^parse error: \(.\+\) at line \(\d\+\), column \(\d\+\)$' let s:pattern = 'parse error: \(.\+\) at line \(\d\+\), column \(\d\+\)$'
function! ale_linters#json#jq#Handle(buffer, lines) abort function! ale_linters#json#jq#Handle(buffer, lines) abort
return ale#util#MapMatches(a:lines, s:pattern, {match -> { return ale#util#MapMatches(a:lines, s:pattern, {match -> {

12
test/handler/test_jq_handler.vader
View File

@@ -16,3 +16,15 @@ Execute (Should parse error correctly):
\ ale_linters#json#jq#Handle(0, [ \ ale_linters#json#jq#Handle(0, [
\ 'parse error: Expected another array element at line 1, column 9' \ 'parse error: Expected another array element at line 1, column 9'
\ ]) \ ])
AssertEqual
\ [
\ {
\ 'lnum': 1,
\ 'col': 9,
\ 'text': 'Expected another array element',
\ }
\ ],
\ ale_linters#json#jq#Handle(0, [
\ 'jq: parse error: Expected another array element at line 1, column 9'
\ ])