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

Merge pull request #3253 from sblask/support-markdownlint-0.19.0-and-0.22.0

Browse Source 
Support markdownlint 0.19.0 and 0.22.0
This commit is contained in:
Horacio Sanson
2020-07-30 09:45:13 +09:00
committed by GitHub
parent 5338dbf680 7cada95683
commit e03e24c091
2 changed files with 72 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
autoload/ale/handlers/markdownlint.vim
View File

@@ -2,15 +2,22 @@
" Description: Adds support for markdownlint " Description: Adds support for markdownlint
function! ale#handlers#markdownlint#Handle(buffer, lines) abort function! ale#handlers#markdownlint#Handle(buffer, lines) abort
let l:pattern=': \(\d*\): \(MD\d\{3}\)\(\/\)\([A-Za-z0-9-]\+\)\(.*\)$' let l:pattern=': \?\(\d\+\)\(:\(\d\+\)\?\)\? \(MD\d\{3}/[A-Za-z0-9-]\+\) \(.*\)$'
let l:output=[] let l:output=[]
for l:match in ale#util#GetMatches(a:lines, l:pattern) for l:match in ale#util#GetMatches(a:lines, l:pattern)
call add(l:output, { let l:result = ({
\ 'lnum': l:match[1] + 0, \ 'lnum': l:match[1] + 0,
\ 'text': '(' . l:match[2] . l:match[3] . l:match[4] . ')' . l:match[5], \ 'code': l:match[4],
\ 'text': l:match[5],
\ 'type': 'W', \ 'type': 'W',
\}) \})
if len(l:match[3]) > 0
let l:result.col = (l:match[3] + 0)
endif
call add(l:output, l:result)
endfor endfor
return l:output return l:output

71
test/handler/test_markdownlint_handler.vader
View File

@@ -4,21 +4,74 @@ Before:
After: After:
call ale#linter#Reset() call ale#linter#Reset()
Execute(The Markdownlint handler should parse output correctly): Execute(The Markdownlint handler should parse pre v0.19.0 output with single digit line correctly):
AssertEqual AssertEqual
\ [ \ [
\ { \ {
\ 'lnum': 1, \ 'lnum': 1,
\ 'text': '(MD002/first-header-h1) First header should be a top level header [Expected: h1; Actual: h2]', \ 'code': 'MD013/line-length',
\ 'type': 'W' \ 'text': 'Line length [Expected: 80; Actual: 114]',
\ },
\ {
\ 'lnum': 298,
\ 'text': '(MD033/no-inline-html) Inline HTML [Element: p]',
\ 'type': 'W' \ 'type': 'W'
\ } \ }
\ ], \ ],
\ ale#handlers#markdownlint#Handle(0, [ \ ale#handlers#markdownlint#Handle(0, [
\ 'README.md: 1: MD002/first-header-h1 First header should be a top level header [Expected: h1; Actual: h2]', \ 'README.md: 1: MD013/line-length Line length [Expected: 80; Actual: 114]'
\ 'README.md: 298: MD033/no-inline-html Inline HTML [Element: p]' \ ])
Execute(The Markdownlint handler should parse pre v0.19.0 output with multi digit line correctly):
AssertEqual
\ [
\ {
\ 'lnum': 100,
\ 'code': 'MD013/line-length',
\ 'text': 'Line length [Expected: 80; Actual: 114]',
\ 'type': 'W'
\ }
\ ],
\ ale#handlers#markdownlint#Handle(0, [
\ 'README.md: 100: MD013/line-length Line length [Expected: 80; Actual: 114]'
\ ])
Execute(The Markdownlint handler should parse post v0.19.0 output with single digit line correctly):
AssertEqual
\ [
\ {
\ 'lnum': 1,
\ 'code': 'MD013/line-length',
\ 'text': 'Line length [Expected: 80; Actual: 114]',
\ 'type': 'W'
\ }
\ ],
\ ale#handlers#markdownlint#Handle(0, [
\ 'README.md:1 MD013/line-length Line length [Expected: 80; Actual: 114]'
\ ])
Execute(The Markdownlint handler should parse post v0.19.0 output with multi digit line correctly):
AssertEqual
\ [
\ {
\ 'lnum': 100,
\ 'code': 'MD013/line-length',
\ 'text': 'Line length [Expected: 80; Actual: 114]',
\ 'type': 'W'
\ }
\ ],
\ ale#handlers#markdownlint#Handle(0, [
\ 'README.md:100 MD013/line-length Line length [Expected: 80; Actual: 114]'
\ ])
Execute(The Markdownlint handler should parse post v0.22.0 output with column correctly):
AssertEqual
\ [
\ {
\ 'lnum': 10,
\ 'col': 20,
\ 'code': 'MD013/line-length',
\ 'text': 'Line length [Expected: 80; Actual: 114]',
\ 'type': 'W'
\ }
\ ],
\ ale#handlers#markdownlint#Handle(0, [
\ 'README.md:10:20 MD013/line-length Line length [Expected: 80; Actual: 114]'
\ ]) \ ])