From e4b3f579fa3c5e4edc8214194cfab4d8cb8aaa6b Mon Sep 17 00:00:00 2001 From: KabbAmine Date: Mon, 10 Oct 2016 15:53:54 +0400 Subject: [PATCH] Echo string format (#76) * Implement an option to configure the echoed message, #48 Via `g:ale_echo_msg_format` where: - `%s` is the error message itself - `%linter%` is the linter name - `%severity` is the severity type e.g let g:ale_echo_msg_fomat = '[%linter%] [%severity%] %s' * Add new options for defining the string used for errors in echoed message `g:ale_echo_msg_error_str` and `g:ale_echo_msg_warning_str` * Change text output of some linters Now that the echoed message can be customized, no need to add the type to the text variable. * Update README & documentation file * Fix some typos * Sort the table of options alphabetically (except echo_msg_x_str options) * Added echo warning str option to the doc --- README.md | 37 ++++++++++++++++++++++++---- ale_linters/coffee/coffeelint.vim | 2 +- ale_linters/yaml/yamllint.vim | 2 +- doc/ale.txt | 39 +++++++++++++++++++++++++++--- img/echo.png | Bin 0 -> 5154 bytes plugin/ale/aaflags.vim | 9 +++++++ plugin/ale/cursor.vim | 21 +++++++++++++++- 7 files changed, 99 insertions(+), 11 deletions(-) create mode 100644 img/echo.png diff --git a/README.md b/README.md index a628e9e..d142f40 100644 --- a/README.md +++ b/README.md @@ -76,20 +76,23 @@ vimrc file for all given linters is as follows: | Option | Description | Default | | ------ | ----------- | ------- | -| `g:ale_linters` | a dictionary of linters to whitelist | _not set_ | -| `g:ale_lint_on_text_changed` | lint while typing | `1` | +| `g:ale_echo_cursor` | echo errors when the cursor is over them | `1` | +| `g:ale_echo_msg_format` | string format to use for the echoed message | `'%s'` | +| `g:ale_echo_msg_error_str` | string used for error severity in echoed message | `'Error'` | +| `g:ale_echo_msg_warning_str` | string used for warning severity in echoed message | `'Warning'` | | `g:ale_lint_delay` | milliseconds to wait before linting | `200` | +| `g:ale_linters` | a dictionary of linters to whitelist | _not set_ | | `g:ale_lint_on_enter` | lint when opening a file | `1` | | `g:ale_lint_on_save` | lint when saving a file | `0` | +| `g:ale_lint_on_text_changed` | lint while typing | `1` | | `g:ale_set_loclist` | set the loclist with errors | `1` | | `g:ale_set_signs` | set gutter signs with error markers | `has('signs')` | | `g:ale_sign_column_always` | always show the sign gutter | `0` | | `g:ale_sign_error` | the text to use for errors in the gutter | `'>>'` | -| `g:ale_sign_warning` | the text to use for warnings in the gutter | `'--'` | | `g:ale_sign_offset` | an offset for sign ids | `1000000` | -| `g:ale_echo_cursor` | echo errors when the cursor is over them | `1` | +| `g:ale_sign_warning` | the text to use for warnings in the gutter | `'--'` | +| `g:ale_statusline_format` | string format to use in statusline flag | `['%d error(s)', '%d warning(s)', 'OK']` | | `g:ale_warn_about_trailing_whitespace` | enable trailing whitespace warnings for some linters | `1` | -| `g:ale_statusline_format` | String format to use in statusline flag | `['%d error(s)', '%d warning(s)', 'OK']` | ### Selecting Particular Linters @@ -155,6 +158,30 @@ let g:ale_statusline_format = ['⨉ %d', '⚠ %d', '⬥ ok'] ![Statusline with issues](img/issues.png) ![Statusline with no issues](img/no_issues.png) + +### Customize echoed message + +There are 3 global options that allow customizing the echoed message. + +- `g:ale_echo_msg_format` where: + * `%s` is the error message itself + * `%linter%` is the linter name + * `%severity` is the severity type +- `g:ale_echo_msg_error_str` is the string used for error severity. +- `g:ale_echo_msg_warning_str` is the string used for warning severity. + +So for example this: + +```vim +let g:ale_echo_msg_error_str = 'E' +let g:ale_echo_msg_error_str = 'W' +let g:ale_echo_msg_fomat = '[%linter%] %s [%severity%]' +``` + +Will give you: + +![Echoed message](img/echo.png) + ## Installation To install this plugin, you should use one of the following methods. diff --git a/ale_linters/coffee/coffeelint.vim b/ale_linters/coffee/coffeelint.vim index aba29c4..972fa25 100644 --- a/ale_linters/coffee/coffeelint.vim +++ b/ale_linters/coffee/coffeelint.vim @@ -27,7 +27,7 @@ function! ale_linters#coffee#coffeelint#Handle(buffer, lines) let line = l:match[1] + 0 let column = 1 let type = l:match[3] ==# 'error' ? 'E' : 'W' - let text = l:match[3] . ': ' . l:match[4] + let text = l:match[4] " vcol is needed to indicate that the column is a character call add(output, { diff --git a/ale_linters/yaml/yamllint.vim b/ale_linters/yaml/yamllint.vim index cb7ca4b..92a4313 100644 --- a/ale_linters/yaml/yamllint.vim +++ b/ale_linters/yaml/yamllint.vim @@ -23,7 +23,7 @@ function! ale_linters#yaml#yamllint#Handle(buffer, lines) let line = match[1] + 0 let col = match[2] + 0 let type = match[3] - let text = printf('[%s]%s', type, match[4]) + let text = match[4] " vcol is Needed to indicate that the column is a character. call add(output, { diff --git a/doc/ale.txt b/doc/ale.txt index bdd2b80..ec2e802 100644 --- a/doc/ale.txt +++ b/doc/ale.txt @@ -1,7 +1,7 @@ -*ale.txt* For Vim version 8.0. Last change: 2016 October 7 +*ale.txt* For Vim version 8.0. Last change: 2016 October 10 *ale* -ALE - Asychronous Lint Engine +ALE - Asynchronous Lint Engine =============================================================================== CONTENTS *ale-contents* @@ -157,7 +157,7 @@ g:ale_set_signs *g:ale_set_signs* When this option is set to `1`, the |sign| column will be populated with signs marking where errors and warnings appear in the file. The - 'ALEErrorSign' and 'ALEWarningSign' highlight groups will be used to provide + `ALEErrorSign` and `ALEWarningSign` highlight groups will be used to provide highlighting for the signs. The text used for signs can be customised with the |g:ale_sign_error| and |g:ale_sign_warning| options. @@ -216,6 +216,39 @@ g:ale_echo_cursor *g:ale_echo_cursor* error at a column nearest to the cursor when the cursor is resting on a line which contains a warning or error. This option can be set to `0` to disable this behaviour. + The format of the message can be customizable in |g:ale_echo_msg_format|. + + +g:ale_echo_msg_format *g:ale_echo_msg_format* + + Type: |String| + Default: `%s` + + This variable defines the format of the echoed message. The `%s` is the + error message itself, and it can contain the following handlers: + - `%linter%` for linter's name + - `%severity%` for the type of severity + Note |`g:ale_echo_cursor`| should be setted to 1 + + +g:ale_echo_msg_error_str *g:ale_echo_msg_error_str* + + Type: |String| + Default: `Error` + + The string used for error severity in the echoed message. + Note |`g:ale_echo_cursor`| should be setted to 1 + Note |`g:ale_echo_msg_format`| should contain the `%severity%` handler + + +g:ale_echo_msg_warning_str *g:ale_echo_msg_warning_str* + + Type: |String| + Default: `Warning` + + The string used for warning severity in the echoed message. + Note |`g:ale_echo_cursor`| should be setted to 1 + Note |`g:ale_echo_msg_format`| should contain the `%severity%` handler g:ale_warn_about_trailing_whitespace *g:ale_warn_about_trailing_whitespace* diff --git a/img/echo.png b/img/echo.png new file mode 100644 index 0000000000000000000000000000000000000000..671a66b26b5a6077963f3e652fbc31fdb0f3f1f2 GIT binary patch literal 5154 zcmZvAc|25a*uKh^HDyaQB|Q=bZCA&w0*$-Pd#7SA>R|5pEa5Yn>=+xucjl3nfvLh>2ZyX{na`e zjn&FwMmS=)I~g}zeNC;8>px+Bo))Kje55E=d954d->%rb3)&k*4i+)ZH8q$zI`&WQ zf0bsxtBHr+1V;IOU*bm@d{Mln3@(3ze10&uh(J#L@gB9ptA|(f?*T3CSNOw6|8vK3 z;WIGazb^m#gj^ZuBVY818U^(Ds6o%fPXXNc#*sMJTWz+Eb#||iI@!TxpR?m619?v!da>ZwX950@vjt9=E9aQH=|(}mw{{YhyEpX2O!3ALQ>$LO?= zQO09>k;dBovp1g$4R9DMbou)b(YIjb=MhWTl#1H zO40|kJv10#qr+h-E1$k&JZqar;VHs=Qj$7_=%Y-<1RDF0DQlNHIBmVM*0RS)BDUCEkAOB_&KHvL zCn|Tw(q}t?gn!(TT2bg_L6b0PjRtk}NiHs)W}OFo&R%Z!PiGV*9VUYi{^f0N;yb>` zzc@Hvf7O#z#>2x?>EI^U+_c`!B>9ijfyX6TTWnd3Od7`6g`OQ4dpnkj@Qi z5VN`Ei=8K8&zzt79doVxjg$M!B^oG2dXdGe0#Arb(b+tZvn3HbRSX;56FubYzmL1i z^q$*V13)kz`l0dWP5cf%H>be!5IYzc;GZmEPD3OTlT${|BQ+$71tdpDj>ga#<)bQt z+x8A*bY+1Yu58dO<5>X#f%(Edd24Zw_;@Wn%#zSEubq~NaM(k8`!wa^|ZLC z8kkfuD~RK*fUwN5xMNxWjH>dLcUx;rc}R<@ov^powFt*qOrKz6CN5V+$qZ~n@yC6{ z$FYOg^~nki*(%s@A%TS;V)2Dh{`o7_W|PZk9%$PX=ET~EyvucB*BF=NGoVOJ1Q}3ejn|0xi5w&Wrefxv0m=8 zeDwTOae4gUc&9vc3pY0x0}w=)Fuw(`ju_6!-zEY*|G~)MY{-W%UZfs5803P%E8U}; z_S1KJx2qO^+7iuD^$c<~_}QP^k^>+k`?EZbpYP9TsH(FxA4W}97YsJpvAu*7B#ZH@vTo(PSp6VO46mu73-(#c(^qT$>XjJF{vu?hkMoVnz|vB& zbsIy-reA<+YqDo8Ko1Z~y7tLp3V~u9dpt|`vLL2JT}6eq*{}(f(Qtf|)AuE|mx|B~ zg;=p>l@pDRcAw~^2s0i4$mu`C0*yk%+?P}}sAqN)b zd2q1NO~SYTr@1`X>|nRjm+{{eLGp!&-s!>iuj&tJYSt6-!QE_sq)@m{iYjw2ewW<{ zZ_o9Pk8GsQ4dLctDwNx*`O^>cH-&}Y$NV*CB0r&@uai9!)DeN%3>WaG(y%~{^2WUK zQ3*WP*^qpOM(N(J8{}%|KBnB!oy4hHPKvNvS~n<`s|NVq+^{^-Wqtmb99Ws;%^vV zVA;H#CT*2fRESoy5c^a~OVWPzQ|5qlnB_6#QD)vqjSw{6+iFD{ND=+>Ib{HoNQ)Mc zs~7$eO`h#*i+Y+mL|)v_K~l(;Y*%NWJl)^*cG>~iXb4O}`ye!x4g8>v;plk%?WV3{ z2q?N`IQM~AfjZBF2a%DMKAM_3ha=i$_;BPeoZZ^oXz-KV^Y7{91>!vd=E0N)mSa%@ z=T#1D<7K$vW+S_YQ6Oj>qOY|b_ZF@0Iz8IL)U4`jM)&QEf$65_4Z@spsGDOGRTG7vN=xpkq|7Y@cN%_!4>%qi z9(`x#zXqYR+{oMgHzyYKtl+Xau3q{U2-J6!@$Ho0)WJKH??wfoyYoZ%FE1}gS?3(E znp>}^j$g$y&lI(n{I@6g)zxM_<@{`xg%S^q@9>NCFk-T;P5n|4d}?!ageF^BWfS#X za>jUU_g2L=XTt|}yxrGNS{yJ9^&%g39;+#4Ecg0H=j)a%w)lO-0GR8XJ8{Qhl9z;$3ABgKmaVYvrz_A)o$94O%8c&J)Yo5;Y-7b zDr*y!M)@!Gbz~*-8@%=-npl-0S9G#O#!y!E%aeV5p+iGj3kyy+v$D_F+ys((9}mx| zCk>ZPO-+gH?hc0RJkJAdzkhgoHHP6xHBB_gYtAI}NmI@!R~F{cK4Wk?x=wOqJ1p&v zf&LR;6e!c^*}~Qcr<Yo$-tJzUFqrsJtcZw%AOO{<~Z~Kdv%+bIf~t*@tIF)oPY8 z4dl&gqLnkR7O^XTa|whDG^wK+&WHJZqT9d~2tDuW?JcVq#>lPTK89%8)gr(qO%NcI z71h#&qI?t zu6rd9ZI1K#j;81-84p@#452PA5kEY-N`uFpIyRLIzM@cHvP?8=k^?*rYhGx71wtTg zkH3$ob6*6N3-jnQyR_I-*8L8byAwy*qDy6 zSbE?J!QJQW+=bhv(Z1JF@jScTIrVz_`jyY0(&;$f4GmT7O?^_o*SN-khpIYf&ic zu71tkec|cb6ZktIf^o>UoT@IGCK}RS)H1(C!!V3|zi$;oDNorlSx-^kBon(SbyKgw zF*pGGHbb|F^z^Hkm{_i6E(B8}UF-el^l|q`c>O{Zje(v<$QYy(-93~mE>-?yo0Sz) zgR&LG!{Vz=Y-`KTdZsXyt?r=;ZAGvX(SmN_=+25t-IB;bYh?kUL^k*6;^5F(YWhP&W-p*g@Vlvjw<($M3#Gw|A3 zNkFA5W1D3)9}(!K;g1pl7;xCDXoy1(%95n1 z$p=S)`_=UH#D_q7UQ%qS;+2s6ZeGHegLZu#|069iZw)!^dY=h#htF92J9~#z-TMCD z6|a6a)Qhf1ROVS)-1I-<-dHgE=(%Njin`9s!ouVx!0P8`&>4IE-FT+|>r9zo%+KTp zIBs@HcQPW8f`_+oVYL006dV~ai75elW+m;EF|*wEJe=(l{w>EY>$f7Fo?+n3fBsZa z(H7V*0ioggXv6ph9fR~s;c~~ur-a+2s3^0agqK=cLr?YZYfzq^E$pncokknjjH7Ew zXYn`b|E08?N_>Ywc~f{=8g1m)AL71U_34Xre->vGu3Vh1Q3cnydztqCSZ_RQ#Vr-N z9wHn4&%EQ}QWUkcXaLQ%us#@3-mLF5f|hhVO_U-v2i6%IbPX=X@iWtWV%SmgjQK2# zCCO(Up+?*R70SLYM;EFfzp1;EoRU%rNcJkU$Jr4V!`&DCU4ipOkt$t%eeHOsH>H`^ z6VgmvQC{tZ1qH(ma4#Sy78eJa2i40XpYh{@H;N4<@^5F`3_pDOY+|@~@XBuwa|49l zmoLdYXEeHM%~GOL$en7ju6|u6rrU}TE6MGtnvT}Fn2L+}#4%}yJ@co}C6L>gtVg5% zFn}JIIDG?MaxVsLYde1N{?2XSGkWIrd)3dzMu1OUNvr}ZC6%BBy!HgBilQwJ8nrFD zMK3G<)iSyn&H@Ihvj^x^0d*ytv%(D^TMY0Ll?QJ_AKSdjcAv z1i_0aO**`T{xgzu+YLGJj`&>|0B#$gI_f<(8FH_N%@d?hoQa@yTV@oC1SUVssznF^szOg6p{3WOY&h)V z>0DtBaBW0*)DYDtU1W&e;dKkJT`@PmEk|93AfhdVvCEhD9tR31Ze9Op0z zZEf5T3Wmb$>@Rpmu696Ue4teT!xD*Mr3L6ZG>@>8N$`;6=i~KJ-cSWVXQPsmQV;Hi zdym`AR#(@Wj_&;;-=@1?)5dQyKL~>ZgtReJEL)Bh%>wj?qN_5ZkCqx@r$O+&*y`~} z1Xt=ZGhG&x`ypLHL>AUj zFooVvyt;g83qe|0j>{z6qi*xx+P1fu?aoktadz}!VWYk|nOkxcD7Z)d$6aCvycZ@O zkm*8L8ek!jxPi6URWQ{2zu&qM f^#9GO-AlK)EX0QLl@`GCBZDfcy)ILD_woM$OJh+r literal 0 HcmV?d00001 diff --git a/plugin/ale/aaflags.vim b/plugin/ale/aaflags.vim index 10d8212..f18dcc4 100644 --- a/plugin/ale/aaflags.vim +++ b/plugin/ale/aaflags.vim @@ -48,3 +48,12 @@ let g:ale_sign_column_always = get(g:, 'ale_sign_column_always', 0) let g:ale_statusline_format = get(g:, 'ale_statusline_format', \ ['%d error(s)', '%d warning(s)', 'OK'] \) + +" String format for the echoed message +" A %s is mandatory +" It can contain 2 handlers: %linter%, %severity% +let g:ale_echo_msg_format = get(g:, 'ale_echo_msg_format', '%s') + +" Strings used for severity in the echoed message +let g:ale_echo_msg_error_str = get(g:, 'ale_echo_msg_error_str', 'Error') +let g:ale_echo_msg_warning_str = get(g:, 'ale_echo_msg_warning_str', 'Warning') diff --git a/plugin/ale/cursor.vim b/plugin/ale/cursor.vim index e0751a8..24f1f18 100644 --- a/plugin/ale/cursor.vim +++ b/plugin/ale/cursor.vim @@ -7,6 +7,23 @@ endif let g:loaded_ale_cursor = 1 +" Return a formatted message according to g:ale_echo_msg_format variable +function! s:GetMessage(linter, type, text) abort + let msg = g:ale_echo_msg_format + let type = a:type ==# 'E' + \ ? g:ale_echo_msg_error_str + \ : g:ale_echo_msg_warning_str + " Capitalize the 1st character + let text = toupper(a:text[0]) . a:text[1:-1] + + " Replace handlers if they exist + for [k, v] in items({'linter': a:linter, 'severity': type}) + let msg = substitute(msg, '\V%' . k . '%', v, '') + endfor + + return printf(msg, text) +endfunction + " This function will perform a binary search to find a message from the " loclist to echo when the cursor moves. function! s:BinarySearch(loclist, line, column) @@ -81,7 +98,9 @@ function! ale#cursor#EchoCursorWarning(...) let index = s:BinarySearch(loclist, pos[1], pos[2]) if index >= 0 - call ale#cursor#TruncatedEcho(loclist[index]['text']) + let l = loclist[index] + let msg = s:GetMessage(l.linter_name, l.type, l.text) + call ale#cursor#TruncatedEcho(msg) else echo endif